adjust metadata loading to address use of API pkgs in Metadata.toml keys

Author: ablaom

src/metadata.jl

@@ -61,16 +61,18 @@ end
 # to define INFO_GIVEN_HANDLE
 function info_given_handle(metadata_file)
     metadata = LittleDict(TOML.parsefile(metadata_file))
-    metadata_given_pkg = decode_dic(metadata)
+    metadata_given_api_pkg = decode_dic(metadata)
 
     # build info_given_handle dictionary:
     ret = Dict{Handle}{Any}()
-    packages = keys(metadata_given_pkg)
-    for pkg in packages
-        info_given_name = metadata_given_pkg[pkg]
+    packages = keys(metadata_given_api_pkg)
+    for api_pkg in packages
+        info_given_name = metadata_given_api_pkg[api_pkg]
         for name in keys(info_given_name)
+            info = info_given_name[name]
+            pkg = info[:package_name]
             handle = Handle(name, pkg)
-            ret[handle] = info_given_name[name]
+            ret[handle] = info
         end
     end
     return ret

test/registry.jl

@@ -20,17 +20,21 @@ using Suppressor
     close(stream)
     registry = dirname(filename) # we need to rename project file to ..../Project.toml
     mv(filename, joinpath(registry, "Project.toml"); force=true)
+    rm(joinpath(registry, "Manifest.toml"); force=true)
 
     # open a new Julia process in which to activate the registry project and attempt to
     # load all models:
     id = only(addprocs(1))
 
     # define the programs to run in that process:
     # 1. To instantiate the registry environment:
+    this_package = joinpath(@__DIR__, "..")
     program1 = quote
         using Pkg
         Pkg.activate($registry)
+        Pkg.develop(path=$this_package) # MLJModels
         Pkg.instantiate()
+        Pkg.status()
         using MLJModels
         !isempty(keys(Pkg.dependencies()))
     end
@@ -42,7 +46,9 @@ using Suppressor
     # remove `@suppress` to debug:
     @test @suppress remotecall_fetch(Main.eval, id, program1)
     @info "Attempting to load all MLJ Model Registry models into a Julia process. "
-    @info "Be patient, this may take five minutes or so..."
+    @info "Be patient, this may take a few minutes ..."
     @test @suppress remotecall_fetch(Main.eval, id, program2)
     rmprocs(id)
 end
+
+true