Skip to content

Commit

Permalink
test(samples): disable samples tests for this repository (#440)
Browse files Browse the repository at this point in the history
Fixes #430.
  • Loading branch information
alexander-fenster authored Sep 20, 2022
1 parent d00839e commit 10aeaaa
Showing 1 changed file with 5 additions and 7 deletions.
12 changes: 5 additions & 7 deletions container-analysis/snippets/test/containerAnalysis.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ let projectId;
let formattedParent;
let formattedNoteName;

describe('Note tests', () => {
describe.skip('Note tests', () => {
before(async () => {
// define projectId and related vars
projectId = await client.getProjectId();
Expand Down Expand Up @@ -236,19 +236,17 @@ describe('Note tests', () => {
);
assert.include(output, 'Occurrence deleted:');
});
it('should delete note', function () {
this.retries(3);

it('should delete note', () => {
const output = execSync(`node deleteNote.js "${projectId}" "${noteId}" `);
assert.include(output, `Note ${formattedNoteName} deleted.`);
// Sometimes the delete note test is failing with the error:
// Error: 5 NOT_FOUND: note with ID "test-note-${uuid}" for project
// ${projectId} does not exist.
// Attempting to work around this issue by retrying a few times.
// DO NOT MERGE. If this works, we should submit an upstream bug.
});
});

describe('polling', () => {
describe.skip('polling', () => {
before(async () => {
// define project id and related vars
projectId = await client.getProjectId();
Expand Down Expand Up @@ -303,7 +301,7 @@ describe('polling', () => {
});
});

describe('pubsub', () => {
describe.skip('pubsub', () => {
before(async () => {
// define project id and related vars
projectId = await client.getProjectId();
Expand Down

0 comments on commit 10aeaaa

Please sign in to comment.