Mirrors/unleash.unleash
1
0
Fork 0
mirror of https://github.com/Unleash/unleash.git synced 2025-10-27 11:02:16 +01:00
Code Issues Projects Releases Wiki Activity

auto disable, and do not serve archived toggle

Browse Source
This commit is contained in:
sveisvei 2017-06-28 17:44:14 +02:00
parent e0e7d919ef
commit 45c2021bd6
2 changed files with 29 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/db/feature-toggle-store.js
View File

@ -47,7 +47,7 @@ class FeatureToggleStore {
return this.db
.first(FEATURE_COLUMNS)
.from(TABLE)
.where({ name })
.where({ name, archived: 0 })
.then(this.rowToFeature);
}
@ -103,10 +103,10 @@ class FeatureToggleStore {
_archiveFeature({ name }) {
return this.db(TABLE)
.where({ name })
.update({ archived: 1 })
.catch(err =>
logger.error('Could not archive feature, error was: ', err)
);
.update({ archived: 1, enabled: 0 })
.catch(err => {
logger.error('Could not archive feature, error was: ', err);
});
}
_reviveFeature({ name }) {

24
test/e2e/api/admin/feature-archive.e2e.test.js
View File

@ -31,6 +31,30 @@ test.serial('revives a feature by name', async t => {
.then(destroy);
});
test.serial(
'archived feature is not accessible via /features/:featureName',
async t => {
t.plan(0);
const { request, destroy } = await setupApp('archive_serial2');
await request
.get('/api/admin/features/featureX')
.set('Content-Type', 'application/json')
.expect(200);
await request
.delete('/api/admin/features/featureX')
.set('Content-Type', 'application/json')
.expect(200);
return request
.get('/api/admin/features/featureX')
.set('Content-Type', 'application/json')
.expect(404)
.then(destroy);
}
);
test.serial('must set name when reviving toggle', async t => {
t.plan(0);
const { request, destroy } = await setupApp('archive_serial');