Skip to content

Ert tests#2289

Merged
npostavs merged 5 commits intodimitri:masterfrom
npostavs:ert-tests
Nov 15, 2015
Merged

Ert tests#2289
npostavs merged 5 commits intodimitri:masterfrom
npostavs:ert-tests

Conversation

@npostavs
Copy link
Copy Markdown
Collaborator

Okay, I'm finding #2264 really needs some tests with it, so it's time to actually get a bit more serious about the ert tests.

Extends/replaces/ closes #969. We'll need to take another look at the failure, but for now I marked it as :expected-failure.

Close #739, it's basically too boring to go over all of the existing issue replicators so I can't see it ever getting done; I'm just moving them to a subdirectory.

Sort of fixes #1691.

@npostavs npostavs force-pushed the ert-tests branch 5 times, most recently from 3805503 to 00e08b1 Compare November 14, 2015 04:39
npostavs and others added 5 commits November 13, 2015 23:39
Also suppress output for passing tests.
rename ert tests file test.el => el-get-tests.el
move el-get-issue-* files to separate dir
Fix compile warnings that snuck in while the test was broken.
npostavs added a commit that referenced this pull request Nov 15, 2015
@npostavs npostavs merged commit 1d68d61 into dimitri:master Nov 15, 2015
@dimitri
Copy link
Copy Markdown
Owner

dimitri commented Nov 15, 2015

Thanks!

@npostavs npostavs deleted the ert-tests branch February 6, 2016 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tests. Like, yes, we should run them! Change test file naming scheme?

3 participants