Add tests - Githubissues

solidusjs / solidus

A simple server that generates pages from JSON and Templates

MIT License

28 stars 7 forks source link

Add tests #23

Closed Fauntleroy closed 11 years ago

Fauntleroy commented 11 years ago

This branch adds some much needed testing to Solidus. In order to facilitate this testing, some introspection into Solidus' functionality was necessary, and a large amount of refactoring was done. The end result is a more extensible, maintainable, and most importantly, testable version of Solidus.

pushred commented 11 years ago

While you're cleaning up can you finally get rid of that .DS_Store?

pushred commented 11 years ago

How about tests for:

[x] Dynamic segments
[x] Preprocessors
[x] Resources
[x] Titles
[x] Clientside templates (invalid)

Fauntleroy commented 11 years ago

@pushred You sure you want a test for clientside templates? Those are currently compiled by the Grunt scripts.

pushred commented 11 years ago

Hmm, I was comparing the tests you had against the readme. Are you going to be decoupling that feature altogether from the server?

Fauntleroy commented 11 years ago

Client side templates currently have nothing to do with Solidus proper, aside from being duct taped on in the form of a grunt task. Here's the task, by the way: https://github.com/SparkartGroupInc/solidus/blob/master/Gruntfile.js#L43

Fauntleroy commented 11 years ago

@pushred anything else?