cweill / gotests

Automatically generate Go test boilerplate from your source code.
Apache License 2.0
4.92k stars 346 forks source link

refactoring: code refactoring #160

Closed butuzov closed 3 years ago

butuzov commented 3 years ago

This code is being refactored, in order to fix race conditions in tests and make the code base more tests-friendly.

Resolves #154

coveralls commented 3 years ago

Coverage Status

Coverage increased (+0.05%) to 95.672% when pulling c76fad4c01748196e45edf9ca3149d6c9bab5669 on butuzov:refactor into 90387e1ec4df7885052899c78aea13cf842c0143 on cweill:develop.

butuzov commented 3 years ago

There are still 2 panics on -race flag.

  1. Test interface embedding: failing with error fatal error: checkptr: converted pointer straddles multiple allocations
  2. Entire testdata directory: failing with data races (because of global variables in templates)

Both of which a kinda not really worth time to invest in a fix.

butuzov commented 3 years ago

@cweill ping.