Merge pull request #200 from Security-Onion-Solutions/kilo

ensure role does not contain the so- prefix
Security-Onion-Solutions · Mar 28, 2023 · 1035a8b · 1035a8b
2 parents 4efa3f5 + 989821c
commit 1035a8b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/html/js/routes/grid.js b/html/js/routes/grid.js
@@ -158,7 +158,7 @@ routes.push({ path: '/grid', name: 'grid', component: {
       return false;
     },
     async gridMemberTest() {
-      const nodeId = this.hideTestConfirm();
+      const nodeId = this.hideTestConfirm().replace('_so-', '_');
       this.$root.startLoading();
       try {
         await this.$root.papi.post('gridmembers/' + nodeId + "/test");

diff --git a/html/js/routes/grid.test.js b/html/js/routes/grid.test.js
@@ -109,7 +109,7 @@ test('canTest', () => {
 test('gridMemberTest', async () => {
 	resetPapi();
 	const mock = mockPapi("post");
-	comp.selectedId = '123';
+	comp.selectedId = 'fwd01_so-sensor';
 	await comp.gridMemberTest();
-	expect(mock).toHaveBeenCalledWith('gridmembers/123/test');
+	expect(mock).toHaveBeenCalledWith('gridmembers/fwd01_sensor/test');
 });