duplicate check for github usernames in registry.yaml

.github/workflows/ci-script-change.yml

@@ -0,0 +1,17 @@
+on:
+  push:
+    paths:
+      - ci/**
+  pull_request:
+    paths:
+      - ci/**
+
+jobs:
+  tests:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Install dependencies
+        run: npm install
+      - name: Unit Tests for CI
+        run: npm run test-ci

ci/check-registry.js

@@ -5,6 +5,7 @@ const path = require('path');
 
 const crypto = require('crypto');
 
+const checkDuplicateGithubUserNames = require('./registry/duplicate-check-ghusername')
 // Create a set to store the hashes of the pageJsonContent strings
 const hashedContents = new Set();
 
@@ -22,6 +23,10 @@ async function validateRegistry() {
     if (!registry || !registry.users) {
         throw new Error('Invalid registry format. The "users" key is missing.');
     }
+
+    // Verify that each github username has only one prefix/domain 
+    checkDuplicateGithubUserNames(registry.users)
+
     // Validate each user
     for (const username in registry.users) {
         const user = registry.users[username];

ci/registry/duplicate-check-ghusername.js

@@ -0,0 +1,12 @@
+async function checkDuplicateGithubUserNames(users){
+    // Verify that each github username has only one prefix/domain 
+    const ghUsernames = Object.values(users).map(user => user.github_username);
+    const duplicatedGhUsernames = ghUsernames.filter((ghUsername, index) => ghUsernames.indexOf(ghUsername) != index)
+
+    if (duplicatedGhUsernames.length > 0) {
+        console.log("duplicated github usernames:",duplicatedGhUsernames.join(","))
+        throw new Error('There are duplicated github usernames in the array');
+    }
+}
+
+module.exports = checkDuplicateGithubUserNames

ci/registry/duplicate-check-ghusername.test.js

@@ -0,0 +1,21 @@
+const checkDuplicateGithubUserNames = require("./duplicate-check-ghusername");
+
+
+
+test("correct scenario, 2 different github usernames, 2 different domains", async () => {
+    await expect(checkDuplicateGithubUserNames({
+        "alice": {"github_username": "alice-gh"},
+        "bob": {"github_username": "bob-gh"},
+    })).resolves.toBeUndefined() // no error
+});
+
+
+test("wrong scenario, 2 same github username, 2 different domains", async () => {
+    expect.assertions(1);
+    await expect(checkDuplicateGithubUserNames({
+        "alice": {"github_username": "alice-gh"},
+        "bob": {"github_username": "alice-gh"},
+    })).rejects.toThrow('There are duplicated github usernames in the array')
+});
+
+