Skip to content

Commit

Permalink
Reenable disabled tess
Browse files Browse the repository at this point in the history
Issue: BB-295
  • Loading branch information
francoisferrand committed Nov 8, 2023
1 parent 0c02e13 commit 54273c6
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 12 deletions.
2 changes: 1 addition & 1 deletion tests/functional/api/retry.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -574,7 +574,7 @@ describe('CRR Retry feature', () => {
});
}, 10000);

it.skip('should get correct data for POST route: /_/crr/failed ' +
it('should get correct data for POST route: /_/crr/failed ' +
'when there are multiple matching keys', done => {
const member = getMember(`test-bucket:test-key:${testVersionId}`);
async.series([
Expand Down
3 changes: 1 addition & 2 deletions tests/functional/ingestion/IngestionProducer.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,7 @@ describe('ingestion producer tests with mock', () => {
emptyAndDeleteVersionedBucket(sourceConfig, done);
});

// skipping because functionality currently not needed
it.skip('should be able to grab list of buckets for each raft session',
it('should be able to grab list of buckets for each raft session',
done => {
this.iProducer._getBuckets('1', (err, res) => {
assert.ifError(err);
Expand Down
3 changes: 1 addition & 2 deletions tests/functional/ingestion/IngestionReader.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -321,8 +321,7 @@ describe('ingestion reader tests with mock', function fD() {
done();
});

// TODO: ZENKO-3420
it.skip('should successfully ingest new bucket with existing object',
it('should successfully ingest new bucket with existing object',
done => {
// update zookeeper status to indicate snapshot phase
const path =
Expand Down
3 changes: 1 addition & 2 deletions tests/functional/lib/BackbeatClient.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ describe('BackbeatClient unit tests with mock server', () => {

afterAll(() => httpServer.close());

// skipping this test because ingestion does not need list bucket per raft
it.skip('should get list of buckets managed by raft session', done => {
it('should get list of buckets managed by raft session', done => {
const destReq = backbeatClient.getRaftBuckets({
LogId: '1',
});
Expand Down
10 changes: 5 additions & 5 deletions tests/unit/lib/queuePopulator/LogReader.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,10 +53,10 @@ describe('LogReader', () => {
});
});

// TODO there is currently no initialization of log offset with
// mongodb backend, re-enable when implementing initial log offset
// fetching with mongodb backend.
it.skip('should start from offset 1 on log consumer readRecords error', done => {
// NOTE: there is currently no initialization of log offset with
// mongodb backend (i.e. it i hard-coded to 1): should be updated
// when implementing initial log offset fetching with mongodb backend.
it('should start from offset 1 on log consumer readRecords error', done => {
const errorLogReader = new LogReader({
logId: 'test-log-reader',
zkClient: zkMock.createClient('localhost:2181'),
Expand All @@ -72,7 +72,7 @@ describe('LogReader', () => {
});
});

it('Should strip metadata v1 prefixes from object entries', done => {
it('should strip metadata v1 prefixes from object entries', done => {
const mockExtension = {
filter: sinon.spy(),
};
Expand Down

0 comments on commit 54273c6

Please sign in to comment.