Skip to content

Make bar tests independent from scales#4686

Merged
etimberg merged 1 commit into
chartjs:masterfrom
andig:bar-tests
Aug 23, 2017
Merged

Make bar tests independent from scales#4686
etimberg merged 1 commit into
chartjs:masterfrom
andig:bar-tests

Conversation

@andig

@andig andig commented Aug 22, 2017

Copy link
Copy Markdown
Contributor

Prepare for changes of #4646 by making some tests less dependent.

@simonbrunel simonbrunel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, we could even hide the title and legend:

options: {
    legend: false,
    title: false
}

@andig any desire to do that for all tests in controller.bar.tests.js :p

@andig

andig commented Aug 23, 2017

Copy link
Copy Markdown
Contributor Author

@andig any desire to do that for all tests in controller.bar.tests.js :p

Looks like something a sane person would write a script for. Let me check after all the help I got here! Its just 384 more failures ;)

@etimberg etimberg merged commit 46c3b24 into chartjs:master Aug 23, 2017
@etimberg

Copy link
Copy Markdown
Member

I think it could be scripted somewhat. A lot of the tests have similar values so once one or two are fixed it might be easier

@andig andig deleted the bar-tests branch August 24, 2017 07:03
yofreke pushed a commit to yofreke/Chart.js that referenced this pull request Dec 30, 2017
exwm pushed a commit to exwm/Chart.js that referenced this pull request Apr 30, 2021
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.

3 participants