Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature tests analysis #208

Merged
merged 21 commits into from
Jan 9, 2019
Merged

Feature tests analysis #208

merged 21 commits into from
Jan 9, 2019

Conversation

flbulgarelli
Copy link
Member

@flbulgarelli flbulgarelli commented Dec 28, 2018

Exposing new interpreter-based test analysis through CLI and JS API.

Also, I opted-in the remaining mandatory options from the CLI.

Pending:

  • Add some tests
  • Actually test it with ghcjs build. Tested in Issue #217 info update #218
  • Add docs
  • Refactor TestAnalyzer, Runner, and Testsmodules whose separation has become less clear after this change.
  • Implement the proposed interpreter-flags. This is mostly outside the scope of this PR, but I wanted to already share my thoughts 😛 See Implement interpreter flags #216
  • Allow test code and extra-code to be specified separately from cli.

@ghost ghost assigned flbulgarelli Dec 28, 2018
@ghost ghost added the pending-review label Dec 28, 2018
@flbulgarelli flbulgarelli force-pushed the feature-tests-analysis branch 2 times, most recently from 8cf51dc to 1d24420 Compare January 1, 2019 16:32
@flbulgarelli
Copy link
Member Author

We should document that now mulang

  • can run tests
  • that only a small subset of imperative programming is supported by now
  • that tests can come with optional extra code
  • that tests can be extracted from source sample
  • that tests can be written in both python and javascript, by now
  • that only a small subset of a test framework is supported by now.

Copy link
Member

@julian-berbel julian-berbel left a comment

Choose a reason for hiding this comment

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

👏

@flbulgarelli
Copy link
Member Author

With latest commit, I converted this PR into a backward-compatible change. See #211

@flbulgarelli flbulgarelli merged commit 448246a into master Jan 9, 2019
@ghost ghost removed the pending-review label Jan 9, 2019
@flbulgarelli flbulgarelli deleted the feature-tests-analysis branch January 9, 2019 21:40
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.

2 participants