fix: should allow revive toggles

src/lib/db/feature-toggle-store.js

@@ -103,7 +103,11 @@ class FeatureToggleStore {
             .first(['project'])
             .from(TABLE)
             .where({ name })
-            .then(r => (r ? r.project : undefined));
+            .then(r => (r ? r.project : undefined))
+            .catch(e => {
+                this.logger.error(e);
+                return undefined;
+            });
     }
 
     async hasFeature(name) {

src/lib/routes/admin-api/archive.ts

@@ -21,7 +21,11 @@ export default class ArchiveController extends Controller {
         this.featureService = featureToggleService;
 
         this.get('/features', this.getArchivedFeatures);
-        this.post('/revive/:name', this.reviveFeatureToggle, UPDATE_FEATURE);
+        this.post(
+            '/revive/:featureName',
+            this.reviveFeatureToggle,
+            UPDATE_FEATURE,
+        );
     }
 
     // eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
@@ -37,9 +41,10 @@ export default class ArchiveController extends Controller {
     // eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
     async reviveFeatureToggle(req, res): Promise<void> {
         const userName = extractUser(req);
+        const { featureName } = req.params;
 
         try {
-            await this.featureService.reviveToggle(req.params.name, userName);
+            await this.featureService.reviveToggle(featureName, userName);
             return res.status(200).end();
         } catch (error) {
             this.logger.error('Server failed executing request', error);

src/test/e2e/stores/feature-toggle-store.e2e.test.js

@@ -24,3 +24,8 @@ test.serial('should not crash for unknown toggle', async t => {
     );
     t.is(project, undefined);
 });
+
+test.serial('should not crash for undefined toggle name', async t => {
+    const project = await featureToggleStore.getProjectId(undefined);
+    t.is(project, undefined);
+});