Add tests for username used to identify the scope

Author: m-tanabe

caliper/database/seeds/EventSeeder.php

@@ -80,5 +80,7 @@ public function run(): void
         Event::create($this->event_factory('\core\event\user_loggedout', 1, 'user', 1));
         Event::create($this->event_factory('\core\event\course_viewed', 1, 'user', 1, null, 'origin1'));
         Event::create($this->event_factory('\core\event\course_viewed', 1, 'user', 1, null, 'origin2'));
+        Event::create($this->event_factory('\core\event\course_viewed', 11, 'user', 1));
+        Event::create($this->event_factory('\core\event\course_viewed', 12, 'user', 1));
     }
 }

caliper/database/seeds/UserSeeder.php

@@ -22,7 +22,17 @@ public function run(): void
                 'lastname' => $faker->lastName,
                 'description' => $faker->sentence(),
                 'email' => $faker->unique()->safeEmail,
+                'alternatename' => $faker->name() . $users[$i % 3],
             ]);
         }
+        App\Models\Moodle\User::create([
+            'auth' => 'lti',
+            'username' => 'username@wellknown.ac.jp',
+            'alternatename' => 'alternatename@wellknown.ac.jp',
+        ]);
+        App\Models\Moodle\User::create([
+            'auth' => 'lti',
+            'username' => 'username@wellknown.ac.jp',
+        ]);
     }
 }

caliper/tests/Feature/EppnTest.php

@@ -4,10 +4,18 @@
 
 use App\Caliper\Traits\Util;
 use App\Models\Eppn;
-use App\Models\Moodle\User;
+use App\Models\Moodle\{
+    Event,
+    User
+};
 use Illuminate\Support\Facades\Config;
 use Tests\TestCase;
 
+use function App\Functions\{
+    compile,
+    expand,
+};
+
 final class EppnTest extends TestCase
 {
     public function setUp(): void
@@ -239,4 +247,55 @@ public function testGetWithoutEppnTableWithLti()
         $this->assertEquals($anonymizedUsername, hash('sha256', $username));
         $this->assertEquals(Eppn::all()->count(), 3);
     }
+
+    public function testGetLtiUsernameWithEppn()
+    {
+        putenv('DB_EPPN=true');
+        $event = Event::where('userid', 11)->first();
+        $user = User::where('id', 11)->first();
+        $interProd = expand($event);
+        $product = compile($interProd);
+        $this->assertEquals(
+            hash('sha256', $user->alternatename),
+            $product->getActor()->getName()
+        );
+        $this->assertEquals(
+            $user->alternatename,
+            $product->getOriginalUsername()
+        );
+    }
+
+    public function testGetLtiUsernameWhenAlternatenameIsNullWithEppn()
+    {
+        putenv('DB_EPPN=true');
+        $event = Event::where('userid', 12)->first();
+        $user = User::where('id', 12)->first();
+        $interProd = expand($event);
+        $product = compile($interProd);
+        $this->assertEquals(
+            '',
+            $product->getActor()->getName()
+        );
+        $this->assertEquals(
+            '',
+            $product->getOriginalUsername()
+        );
+    }
+
+    public function testGetLtiUsernameWithoutEppn()
+    {
+        putenv('DB_EPPN=false');
+        $event = Event::where('userid', 11)->first();
+        $user = User::where('id', 11)->first();
+        $interProd = expand($event);
+        $product = compile($interProd);
+        $this->assertEquals(
+            hash('sha256', $user->username),
+            $product->getActor()->getName()
+        );
+        $this->assertEquals(
+            $user->username,
+            $product->getOriginalUsername()
+        );
+    }
 }

caliper/tests/Feature/ExcludedOriginsTest.php

@@ -22,9 +22,9 @@ public function testExcludedOriginsWithParams()
             'execution_logs',
             [
                 'id' => 1,
-                'translated' => 13,
+                'translated' => 15,
                 'failed' => 1,
-                'last_id' => 17,
+                'last_id' => 19,
             ],
             'log'
         );
@@ -38,9 +38,9 @@ public function testExcludedOriginsWithoutParams()
             'execution_logs',
             [
                 'id' => 1,
-                'translated' => 15,
+                'translated' => 17,
                 'failed' => 1,
-                'last_id' => 17,
+                'last_id' => 19,
             ],
             'log'
         );

caliper/tests/Feature/GetLrsTest.php

@@ -14,6 +14,7 @@ public function testGetEppnScope()
         $test = new MockEvent('user@testscope');
         $test2 = new MockEvent('user@test.ac.jp');
         $test3 = new MockEvent('noeppn');
+        $test4 = new MockEvent('');
 
         $mockConfig = [
             "default" => "default key",
@@ -27,6 +28,7 @@ public function testGetEppnScope()
         $this->assertEquals($tenants['default'], get_lrs($test->getUserName()));
         $this->assertEquals($tenants['test.ac.jp'], get_lrs($test2->getUserName()));
         $this->assertEquals($tenants['default'], get_lrs($test3->getUserName()));
+        $this->assertEquals($tenants['default'], get_lrs($test4->getUserName()));
     }
 }
 

caliper/tests/Feature/LoggingTest.php

@@ -21,9 +21,9 @@ public function testExecutionLog()
             'execution_logs',
             [
                 'id' => 1,
-                'translated' => 15,
+                'translated' => 17,
                 'failed' => 1,
-                'last_id' => 17,
+                'last_id' => 19,
             ],
             'log'
         );