Skip to content

[BUG] test/util.js is using assert incorrectly #852

@jdmarshall

Description

@jdmarshall

Describe the bug

There is at least one test in here that is incorrect and they are all failing silently due to using

  assert(a, b);

instead of

  assert.equal(a, b);

This test in particular is broken:

      process.argv = ['--FROMARG=in-the-argv'];
      assert(initParam('FROMARG', 'mydefault'), 'in-the-argv');
      assert(config.util.getEnv('FROMARG'), 'in-the-argv');

Because we ignore argv[0] and argv[1]

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions