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

Better spec data #32

Closed
wants to merge 1 commit into from
Closed

Better spec data #32

wants to merge 1 commit into from

Conversation

jethrodaniel
Copy link
Collaborator

@jethrodaniel jethrodaniel commented Sep 8, 2019

Adds some spec data (copyright free), and updates the specs.

Removes all of the should syntax in our specs, and replaces them with the more common/modern/supported expect syntax.

Moves all the hardcoded values from the specs into the texts themselves, which are in yaml.


As referenced in #24 (comment)

Resolves #23

@jethrodaniel jethrodaniel marked this pull request as ready for review September 8, 2019 20:44
This removes all the hard coding from the specs, and puts the text
metrics and scores with the texts themselves.

adds activesupport dev dependency, to symbolize the yaml keys the lazy
way.
@jethrodaniel
Copy link
Collaborator Author

What'd I'd really like to do here is to have this gem tested against the same fixtures that other libraries of this sort use.

Ideally, all the yaml fixtures would get migrated out to a single repo, which would be more useful for more people.

Closing this as such.

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.

Fix SMOG index spec
1 participant