mirror of
				https://github.com/explosion/spaCy.git
				synced 2025-11-04 09:57:26 +03:00 
			
		
		
		
	## Description Related issues: #2379 (should be fixed by separating model tests) * **total execution time down from > 300 seconds to under 60 seconds** 🎉 * removed all model-specific tests that could only really be run manually anyway – those will now live in a separate test suite in the [`spacy-models`](https://github.com/explosion/spacy-models) repository and are already integrated into our new model training infrastructure * changed all relative imports to absolute imports to prepare for moving the test suite from `/spacy/tests` to `/tests` (it'll now always test against the installed version) * merged old regression tests into collections, e.g. `test_issue1001-1500.py` (about 90% of the regression tests are very short anyways) * tidied up and rewrote existing tests wherever possible ### Todo - [ ] move tests to `/tests` and adjust CI commands accordingly - [x] move model test suite from internal repo to `spacy-models` - [x] ~~investigate why `pipeline/test_textcat.py` is flakey~~ - [x] review old regression tests (leftover files) and see if they can be merged, simplified or deleted - [ ] update documentation on how to run tests ### Types of change enhancement, tests ## Checklist <!--- Before you submit the PR, go over this checklist and make sure you can tick off all the boxes. [] -> [x] --> - [x] I have submitted the spaCy Contributor Agreement. - [x] I ran the tests, and all new and existing tests passed. - [ ] My changes don't require a change to the documentation, or if they do, I've added all required information.
		
			
				
	
	
		
			30 lines
		
	
	
		
			781 B
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			30 lines
		
	
	
		
			781 B
		
	
	
	
		
			Python
		
	
	
	
	
	
# coding: utf-8
 | 
						|
from __future__ import unicode_literals
 | 
						|
 | 
						|
 | 
						|
def test_en_simple_punct(en_tokenizer):
 | 
						|
    text = "to walk, do foo"
 | 
						|
    tokens = en_tokenizer(text)
 | 
						|
    assert tokens[0].idx == 0
 | 
						|
    assert tokens[1].idx == 3
 | 
						|
    assert tokens[2].idx == 7
 | 
						|
    assert tokens[3].idx == 9
 | 
						|
    assert tokens[4].idx == 12
 | 
						|
 | 
						|
 | 
						|
def test_en_complex_punct(en_tokenizer):
 | 
						|
    text = "Tom (D., Ill.)!"
 | 
						|
    tokens = en_tokenizer(text)
 | 
						|
    assert tokens[0].idx == 0
 | 
						|
    assert len(tokens[0]) == 3
 | 
						|
    assert tokens[1].idx == 4
 | 
						|
    assert len(tokens[1]) == 1
 | 
						|
    assert tokens[2].idx == 5
 | 
						|
    assert len(tokens[2]) == 2
 | 
						|
    assert tokens[3].idx == 7
 | 
						|
    assert len(tokens[3]) == 1
 | 
						|
    assert tokens[4].idx == 9
 | 
						|
    assert len(tokens[4]) == 4
 | 
						|
    assert tokens[5].idx == 13
 | 
						|
    assert tokens[6].idx == 14
 |