search

joshgoebel / wren-testie

Simple and beautiful testing for Wren
MIT License
8 stars 2 forks source link

Add a testing script #6

Closed glennj closed 3 years ago

glennj commented 3 years ago

Requires Process.chdir(_) from wren-console and Strings.globMatch(_, _) from wren-essentials
Latest iteration no longer has these requirements.

joshgoebel commented 3 years ago

Ok, now we have an issue where this wouldn't just work with the release version of wrenc... is it's the script job to check the runtime version, or should the package manager handle that? I worry someone doing development of the library isn't using the package manager though so it would never flag since it was not being used for dev work...

I'm wondering if we should have some sort of Runtime.assertVersion in wren-console itself perhaps - since we're still so young and changing?

glennj commented 3 years ago

I'm wondering if we should have some sort of Runtime.assertVersion in wren-console itself perhaps - since we're still so young and changing?

I'd say it's the script's job since wren is not very introspective, and that assertVersion is a good idea.

joshgoebel commented 3 years ago

https://github.com/joshgoebel/wren-console/pull/15/files

joshgoebel commented 3 years ago

Oh that might be nice. Excited to try this later.