Skip to content

Defeat mocha - #21

Open
Munter wants to merge 1 commit into
auchenberg:masterfrom
Munter:mocha
Open

Defeat mocha#21
Munter wants to merge 1 commit into
auchenberg:masterfrom
Munter:mocha

Conversation

@Munter

@Munter Munter commented Oct 9, 2015

Copy link
Copy Markdown
Contributor

Defeat mocha UI's:

  • BDD
  • TDD
  • QUnit
  • Exports
  • Require

@watson

watson commented Oct 10, 2015

Copy link
Copy Markdown
Collaborator

@Munter Nice - I started on this, but gave up after looking at the API 😉

Comment thread mocha/exports.js

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change to require('../test/_fake-ci')
see #17 for ref

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Guria Thanks for helping out on the repo 😄 We've added you as a collaborator! 🎉

@drewnichols

Copy link
Copy Markdown

Shouldn't your travis build always be green?

@watson

watson commented Oct 12, 2015

Copy link
Copy Markdown
Collaborator

@drewnichols haha yeah - we should eat our own dog food as they say ;)

@Guria

Guria commented Oct 13, 2015

Copy link
Copy Markdown
Collaborator

@drewnichols @watson good point, but then we can miss a problem which can lead to failed tests of our users

@watson

watson commented Oct 13, 2015

Copy link
Copy Markdown
Collaborator

@Guria true - was only joking of cause 😉 But it is kind of funny that we have so good test coverage in a project aimed at not testing 😜

@drewnichols

Copy link
Copy Markdown

I don’t think the aim is to not test. It’s just to ensure that the our bosses never see the tests fail.

Drew Nichols

VP of Engineering

LiftForward, Inc.

261 Madison Avenue, 9th Floor

New York, NY 10016

917-473-3006

On Tue, Oct 13, 2015 at 8:40 AM, Thomas Watson Steen
notifications@github.com wrote:

@Guria true - was only joking of cause 😉 But it is kind of funny that we have so good test coverage in a project aimed at not testing 😜

Reply to this email directly or view it on GitHub:
#21 (comment)

@RussellBradley

Copy link
Copy Markdown

Wait... this isn't green.
chuck norris

@nikole-dunixi

Copy link
Copy Markdown

This is the most meta thing ever.

@battlesnake

Copy link
Copy Markdown

Checks failed for this PR. Perhaps consider installing Volkswagen?

@drewnichols

Copy link
Copy Markdown

any chance we can close this so it's off my view?

@Munter

Munter commented Apr 1, 2017

Copy link
Copy Markdown
Contributor Author

We can disable the tests for the exports and require interfaces. It won't be a full defeat, but I've never seen even a single project use those two interfaces. Everyone seems to be using bdd or tdd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants