Merge pull request #7 from grafana/6-fix-detection-of-new-extensions

fix detection of new extensions

.github/workflows/distro.yml

@@ -143,7 +143,7 @@ jobs:
           fi
 
       - name: Build Distro
-        uses: grafana/[email protected].5
+        uses: grafana/[email protected].6
         if: ${{ steps.latest.outputs.today != 'true' }}
         id: build
         with:

.gitignore

@@ -1,4 +1,5 @@
 /build
 /dist
 /recent.json
-/k6dist
+/k6dist
+/coverage.txt

internal/registry/registry.go

@@ -109,16 +109,16 @@ func loadRegistryHTTP(ctx context.Context, source string) (Registry, error) {
 
 // AddLatest adds latest versions to extensions.
 func (reg Registry) AddLatest(modules Modules) bool {
-	dict := make(map[string]*Extension, len(reg))
+	regAsMap := make(map[string]*Extension, len(reg))
 
 	for idx := range reg {
-		dict[reg[idx].Module] = &reg[idx]
+		regAsMap[reg[idx].Module] = &reg[idx]
 	}
 
 	changed := false
 
 	for _, mod := range modules {
-		ext, found := dict[mod.Path]
+		ext, found := regAsMap[mod.Path]
 		if found {
 			ext.Versions[1] = mod.Version
 			changed = changed || (ext.Versions[0] != mod.Version)
@@ -127,5 +127,21 @@ func (reg Registry) AddLatest(modules Modules) bool {
 		changed = changed || !found
 	}
 
-	return changed
+	if changed {
+		return true
+	}
+
+	modulesAsMap := make(map[string]*Module, len(modules))
+
+	for idx := range modules {
+		modulesAsMap[modules[idx].Path] = &modules[idx]
+	}
+
+	for _, ext := range reg {
+		if _, found := modulesAsMap[ext.Module]; !found {
+			panic("ok")
+		}
+	}
+
+	return false
 }

releases/v0.1.6.md

@@ -0,0 +1,5 @@
+🎉 k6dist `v0.1.6` is here!
+
+**Fix detection of new extensions**
+
+Change detection now correctly detects new extensions released since the previous release.