Merge pull request #469 from mitodl/jkachel/fix-channel-lookup

Make sure there's a channel for Doof to talk to before adding it to the list of repos

Author: jkachel

lib.py

@@ -344,7 +344,7 @@ def load_repos_info(channel_lookup):
             versioning_strategy=repo_info.get("versioning_strategy", FILE_VERSION),
         )
         for repo_info in repos_info["repos"]
-        if repo_info.get("repo_url")
+        if repo_info.get("repo_url") and repo_info.get("channel_name") and channel_lookup.get(repo_info.get("channel_name"))
     ]
 
     # some basic validation for sanity checking

lib_test.py

@@ -211,6 +211,21 @@ async def test_load_repos_info(mocker):
                     "packaging_tool": "none",
                     "versioning_strategy": FILE_VERSION,
                 },
+                {
+                    "name": "bad-no-channel",
+                    "repo_url": "https://github.com/mitodl/ocw-hugo-projects.git",
+                    "project_type": "library",
+                    "packaging_tool": "none",
+                    "versioning_strategy": FILE_VERSION,
+                },
+                {
+                    "name": "bad-nonexistant-channel",
+                    "repo_url": "https://github.com/mitodl/ocw-hugo-projects.git",
+                    "channel_name": "ghost-channel",
+                    "project_type": "library",
+                    "packaging_tool": "none",
+                    "versioning_strategy": FILE_VERSION,
+                },
             ]
         },
     )