screepers / screeps-typescript-starter

Starter kit for TypeScript-based Screeps AI codes.
https://screepers.gitbook.io/screeps-typescript-starter/
The Unlicense
441 stars 318 forks source link

Integration testing cleanup #124

Closed pyrodogg closed 4 years ago

pyrodogg commented 4 years ago

The goal is to keep the framework for integration testing but not require screeps-server-mockup by default. Instead, to instruct users how to install the dependency themselves.

brisberg commented 4 years ago

Should we remove the files under test/integration and make those optional too?

Right now they don't even compile without the screeps-server-mockup dependency.

djD-REK commented 4 years ago

Looks ready to merge to me :)

@brisberg I think the test files can stay, since you won't be running them without following the steps in the docs...

djD-REK commented 4 years ago

Alternatively, what would you think of just making the screeps-server-mockup dependency required for this package? Too much? 🔢

pyrodogg commented 4 years ago

@djD-REK Thanks for bringing this up again. I stalled when considering how much further it could go and forgot that the basic test script is currently broken on master.