It looks like these three strategies generate a single value. Shouldn't we define an auxiliary function and use it with [hypothesis.strategies.builds](https://hypothesis.readthedocs.io/en/latest/data.html#hypothesis.strategies.builds)?

Or maybe make them composite strategies that take bytes as argument and build the hashes from there?

Rename to something like content_sha1s_with_ctags?

I don't understand this strategy.

An example of what you proposed would be great ;)

Antoine most probably elected this as the single revision having an ancestor relation as a building step.
And either forget to work more on this or did not find a way to do better (found other revision like this ;)

The docstring should be fixed to remove the plural on the sentence returning revisions.

No need to have the SWH prefix everywhere, i tried to remove those redundant prefixes in other repositories.

In order for the tests to not take too long to execute, I have limited the maximum number of examples generated by hypothesis to 1.
I also tested to use the from_regex and text strategy from hypothesis to generate the hashes but I ended up
having only string full of '0' (due to the restriction on the numbers of examples).

So that's why I implemented this way.

ack

Yes, I harcoded some revision ids for the tests input by looking at the following git log ouput:

✔ ~/dev/highlightjs-line-numbers.js [master|✔] 
11:58 $ git log --oneline --graph
*   4ff78fc Merge pull request #52 from wcoder/dev
|\  
| * 00356ff Update version
| * f4a0634 Fix whitespaces for not modified sources
| * 9029d20 Add support nohighlight
|/  
*   f1cdbc3 Merge pull request #49 from wcoder/dev
|\  
| * 35dcbb3 Update version
| *   d7c52cd Merge branch 'vai0-master' into dev
| |\  
| | *   346fb4d Merge branch 'master' of https://github.com/vai0/highlightjs-line-numbers.js into vai0-master
| | |\  
| | | * f3f53fe Delete highlightjs-line-numbers.min.js
| | | * 0081bb8 update dist
| | | *   4692cfd Merge pull request #1 from vai0/add-readyState
| | | |\  
| |_|/ /  
|/| | |   
| | | * 1b77273 add forgotten readyState
| |_|/  
|/| |   
| * |   0e411b1 Merge pull request #48 from anlambert/fix-duplicateMultilineNode-empty-lines
| |\ \  
| | |/  
| |/|   
| | * 708a35e Update highlightjs-line-numbers.js
| | * 1cf33c8 Fix output when hljs multi-line element contains an empty line
| |/  
|/|   
| *   aa5ebf1 Merge branch 'master' into dev
| |\  
| |/  
|/|   
* | e03ee42 Update issue templates
* | 5bb5a73 Create CODE_OF_CONDUCT.md
* | c0519a1 Update README.md
* | 8c67b55 Update README.md
* |   675285e Merge pull request #33 from wcoder/dev
|\ \  
| | * 1b24c7d Update version
| |/  
| * a007797 Update README.md
| * 59f905d Fix for the last line if empty
| * 352a5ba Add fix
|/  
*   4ab966b Merge pull request #30 from wcoder/bugfixes/multiline-nodes

I agree that it could be generated programmatically, will look into it.

Ack, I will make the changes in master first and rebase.

well seen!

Quoting hypothesis doc [1]:

The decorated function has the initial argument removed from the list, but will accept all the others in the expected order. Defaults are preserved.

[1] https://hypothesis.readthedocs.io/en/latest/data.html#composite-strategies