youtube / spitfire

A high-performance Python template language
BSD 3-Clause "New" or "Revised" License
404 stars 58 forks source link

Rename unit test and benchmark files #18

Closed nicksay closed 8 years ago

nicksay commented 8 years ago

This changes unit tests from:

module1.py
module2.py
tests/test_module1.py
tests/test_module2.py

to

module1.py
module1_test.py
module2.py
module2_test.py

Benchmarks are changed from:

tests/benchmark_module.py

to

tests/module_benchmark.py
awbraunstein commented 8 years ago

Should benchmarks be in the tests folder? Maybe the folder should be benchmarks/

nicksay commented 8 years ago

There's also the https://github.com/nicksay/spitfire/tree/master/tests/perf folder that's currently outside the code tree

awbraunstein commented 8 years ago

okay. I'm good with a test folder.

lgtm

nicksay commented 8 years ago

So, you're saying create tests/benchmarks?

awbraunstein commented 8 years ago

yeah. tests/benchmarks sounds good.

nicksay commented 8 years ago

Updated with the move:

spitfire/runtime/tests/benchmark_filter_function.py
→
tests/benchmarks/filter_function_benchmark.py
awbraunstein commented 8 years ago

lgtm