Fix automated test suite - #443
Conversation
|
💪 Holy champ! What the what?! 🤯 |
2ef99b2 to
d7d53ba
Compare
|
@bainco , very cool that you did this! 😎 🙇♂️ I fixed the befuddling text that you sidelined for the Interstitial Diffusion model. The logic behind the test is as follows: A View widget's width is computable by However, in the So, indeed, I think this test did actually catch a very minor bug. It seems that the widget's stated height and width were not factoring in the border. I've fixed it in this PR now, and everything looks uncontroversial to me, so I'll merge this and rebase your other PRs onto it. |
Some quick fixes to get the automated test suite working again:
srandresourceextensionsrand thespatstatpackage on the build environment to supportsrextensionssrextension:Code Examples/Extensions Example/sr/plotting.nlogoxtoCode Examples/Extensions Example/sr/Plotting.nlogoxto match.gitignoreentries (though perhaps this should be renamed toPlotting Examplesince all the other example models match that nomenclature)Example 1.nlogoxinsrexamplesBoxplot Exampleexemption from "no code in INFO TABs" test3D MazeNote: there's a weird Graphics Environment exception popping up in the actions logs (
Exception in thread "AWT-EventQueue-0" java.awt.HeadlessException). Not sure what that's all about but worth investigating.