microsoft / microsoft/TypeScript

Move test files and baselines into subdirectories

Ouverte
#42,496 1 commentaire 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Experience Enhancement Suggestion
Langage dominant
Go
Étoiles
111k
Forks
14.3k
Merge moyen
1 j 19 h
PR mergées (30 j)
117

Description

Bug Report

VSCode can't really handle the number of files in the tests/cases/compiler directory (5386) or tests/baselines/reference (36655). It makes working with those directories very awkward!

There's plenty of scope for subdirectories, e.g.

anyAndUnknownHaveFalsyComponents.ts
anyAsReturnTypeForNewOnCall.ts
anyDeclare.ts
anyIdenticalToItself.ts
anyIndexedAccessArrayNoException.ts
anyInferenceAnonymousFunctions.ts
anyIsAssignableToObject.ts
anyIsAssignableToVoid.ts
anyMappedTypesError.ts
anyPlusAny1.ts
argsInScope.ts
arguments.ts
argumentsAsPropertyName.ts
argumentsAsPropertyName2.ts
argumentsBindsToFunctionScopeArgumentList.ts
argumentsObjectCreatesRestForJs.ts
argumentsObjectIterator01_ES5.ts
argumentsObjectIterator01_ES6.ts
argumentsObjectIterator02_ES5.ts
argumentsObjectIterator02_ES6.ts
argumentsObjectIterator03_ES5.ts
argumentsObjectIterator03_ES6.ts
argumentsReferenceInConstructor1_Js.ts
argumentsReferenceInConstructor2_Js.ts
argumentsReferenceInConstructor3_Js.ts
argumentsReferenceInConstructor4_Js.ts
argumentsReferenceInConstructor5_Js.ts
argumentsReferenceInConstructor6_Js.ts
argumentsReferenceInConstructor7_Js.ts
argumentsReferenceInMethod1_Js.ts
argumentsReferenceInMethod2_Js.ts
argumentsReferenceInMethod3_Js.ts
argumentsReferenceInMethod4_Js.ts
argumentsReferenceInMethod5_Js.ts
argumentsReferenceInMethod6_Js.ts
argumentsReferenceInMethod7_Js.ts
argumentsUsedInObjectLiteralProperty.ts
arithAssignTyping.ts
arithmeticOnInvalidTypes.ts
arithmeticOnInvalidTypes2.ts
arityErrorRelatedSpanBindingPattern.ts
arrayAssignmentTest1.ts
arrayAssignmentTest2.ts
arrayAssignmentTest3.ts
arrayAssignmentTest4.ts
arrayAssignmentTest5.ts
arrayAssignmentTest6.ts

Could be in these subdirectories

any
arguments
arithmetic
arity
arrayAssignment

I think that would cut down the number of subdirectories to something like 1000 which is much more reasonable. It could be a mostly automated change since the tests are named nicely and in most cases you could just group them by their first word (or maybe even first 1/2 letters?).

🔎 Search Terms

large number many files test cases baselines

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par examiner les hypothèses de structure et de chemins autour de tests/cases/compiler et tests/baselines/reference, notamment la manière dont les tests du compilateur localisent leurs baselines. Regroupez les fichiers de test et les baselines correspondantes dans des sous-répertoires cohérents, puis vérifiez que la suite de tests du compilateur découvre toujours les tests et correspond à chaque baseline après le déplacement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
typescript
Domaine
testing
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.