remove testSpecifyPackageNamesUnderTest since it uses RPackage and ot…

…hers and I do not see the value of the tests
pillar-markup · Jul 19, 2024 · 610ca03 · 610ca03
1 parent 5fd8b06
commit 610ca03
Show file tree

Hide file tree

Showing 37 changed files with 667 additions and 609 deletions.
diff --git a/src/Pillar-Tests-ExporterCore/ManifestPillarTestsExporterCore.class.st b/src/Pillar-Tests-ExporterCore/ManifestPillarTestsExporterCore.class.st
@@ -2,7 +2,9 @@
 I store metadata for this package. These meta data are used by other tools such as the SmalllintManifestChecker and the critics Browser
 "
 Class {
-	#name : #ManifestPillarTestsExporterCore,
-	#superclass : #PackageManifest,
-	#category : #'Pillar-Tests-ExporterCore-Manifest'
+	#name : 'ManifestPillarTestsExporterCore',
+	#superclass : 'PackageManifest',
+	#category : 'Pillar-Tests-ExporterCore-Manifest',
+	#package : 'Pillar-Tests-ExporterCore',
+	#tag : 'Manifest'
 }
diff --git a/src/Pillar-Tests-ExporterCore/MicTransformDocumentTest.class.st b/src/Pillar-Tests-ExporterCore/MicTransformDocumentTest.class.st
@@ -1,17 +1,19 @@
 Class {
-	#name : #MicTransformDocumentTest,
-	#superclass : #TestCase,
+	#name : 'MicTransformDocumentTest',
+	#superclass : 'TestCase',
 	#instVars : [
 		'filesystem',
 		'builder',
 		'project',
 		'file',
 		'aPRPDFDocument'
 	],
-	#category : #'Pillar-Tests-ExporterCore-Base'
+	#category : 'Pillar-Tests-ExporterCore-Base',
+	#package : 'Pillar-Tests-ExporterCore',
+	#tag : 'Base'
 }
 
-{ #category : #running }
+{ #category : 'running' }
 MicTransformDocumentTest >> generateFilesystemExample [
 
 	| file |
@@ -26,7 +28,7 @@ Pharo is cool
 
 ]
 
-{ #category : #initialization }
+{ #category : 'initialization' }
 MicTransformDocumentTest >> setUp [
 	super setUp.
 	filesystem := FileSystem memory.
@@ -40,7 +42,7 @@ MicTransformDocumentTest >> setUp [
 		target: PRPdfOutput new.
 ]
 
-{ #category : #initialization }
+{ #category : 'initialization' }
 MicTransformDocumentTest >> testTransformDocumentFor [
 
 	| parsedDocument transformedDocument |

diff --git a/src/Pillar-Tests-ExporterCore/PRAbstractOutputDocumentTest.class.st b/src/Pillar-Tests-ExporterCore/PRAbstractOutputDocumentTest.class.st
@@ -1,10 +1,12 @@
 Class {
-	#name : #PRAbstractOutputDocumentTest,
-	#superclass : #PRTestWithMemoryFileSystem,
-	#category : #'Pillar-Tests-ExporterCore-Base'
+	#name : 'PRAbstractOutputDocumentTest',
+	#superclass : 'PRTestWithMemoryFileSystem',
+	#category : 'Pillar-Tests-ExporterCore-Base',
+	#package : 'Pillar-Tests-ExporterCore',
+	#tag : 'Base'
 }
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRAbstractOutputDocumentTest >> testParseInput [
 
 	| doc |

diff --git a/src/Pillar-Tests-ExporterCore/PRAnchorOfSectionTest.class.st b/src/Pillar-Tests-ExporterCore/PRAnchorOfSectionTest.class.st
@@ -1,15 +1,17 @@
 Class {
-	#name : #PRAnchorOfSectionTest,
-	#superclass : #PRNodeTransformerTest,
-	#category : 'Pillar-Tests-ExporterCore-Transformers'
+	#name : 'PRAnchorOfSectionTest',
+	#superclass : 'PRNodeTransformerTest',
+	#category : 'Pillar-Tests-ExporterCore-Transformers',
+	#package : 'Pillar-Tests-ExporterCore',
+	#tag : 'Transformers'
 }
 
-{ #category : #private }
+{ #category : 'private' }
 PRAnchorOfSectionTest >> actualClass [
 	^ PRAnchorOfSection
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRAnchorOfSectionTest >> testAnchorWithSimpleTitle [
 
 	| input expected config |
@@ -27,7 +29,7 @@ PRAnchorOfSectionTest >> testAnchorWithSimpleTitle [
 	self assert: (self executePhase: input with: config) equals: expected
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRAnchorOfSectionTest >> testAnchorWithSimpleTitleAndAnchorNumberName [
 	| input expected config |
 	config := self configuration.
@@ -48,7 +50,7 @@ PRAnchorOfSectionTest >> testAnchorWithSimpleTitleAndAnchorNumberName [
 	self assert: (self executePhase: input with: config) equals: expected
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRAnchorOfSectionTest >> testAnchorWithSimpleTitleAndAnchorWithSameName [
 	| input expected config |
 	config := self configuration.
@@ -70,7 +72,7 @@ PRAnchorOfSectionTest >> testAnchorWithSimpleTitleAndAnchorWithSameName [
 	self assert: (self executePhase: input with: config) equals: expected
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRAnchorOfSectionTest >> testAnchorWithTwoSameTitle [
 	| input expected config |
 	config := self configuration.
@@ -88,7 +90,7 @@ PRAnchorOfSectionTest >> testAnchorWithTwoSameTitle [
 	self assert: (self executePhase: input with: config) equals: expected
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRAnchorOfSectionTest >> testAnchorWithoutTitle [
 	| input config |
 	config := self configuration.

diff --git a/src/Pillar-Tests-ExporterCore/PRBuildRootMainStrategyTest.class.st b/src/Pillar-Tests-ExporterCore/PRBuildRootMainStrategyTest.class.st
@@ -1,24 +1,26 @@
 Class {
-	#name : #PRBuildRootMainStrategyTest,
-	#superclass : #PRBuildStrategyTest,
-	#category : #'Pillar-Tests-ExporterCore-Base'
+	#name : 'PRBuildRootMainStrategyTest',
+	#superclass : 'PRBuildStrategyTest',
+	#category : 'Pillar-Tests-ExporterCore-Base',
+	#package : 'Pillar-Tests-ExporterCore',
+	#tag : 'Base'
 }
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRBuildRootMainStrategyTest >> testFilesToBuildOnCorrect [
 	| outputProject |
 	outputProject := PRBuildRootMainStrategy new filesToBuildOn: project.
 	self assert: outputProject first class equals: PRPillarInputDocument.
 	self assert: outputProject first file path equals: Path / 'index.pillar'
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRBuildRootMainStrategyTest >> testFilesToBuildOnErrorNoFile [
 	project baseDirectory: ((project baseDirectory / 'index.pillar') delete; parent).
 	self should: [ PRBuildRootMainStrategy new filesToBuildOn: project ] raise: Error
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRBuildRootMainStrategyTest >> testFilesToBuildOnErrorSeveralFiles [
 	project baseDirectory: ((project baseDirectory / 'chap3.pillar') ensureCreateFile; parent).
 	self should: [ PRBuildRootMainStrategy new filesToBuildOn: project ] raise: Error

diff --git a/src/Pillar-Tests-ExporterCore/PRBuildStrategyTest.class.st b/src/Pillar-Tests-ExporterCore/PRBuildStrategyTest.class.st
@@ -1,13 +1,15 @@
 Class {
-	#name : #PRBuildStrategyTest,
-	#superclass : #TestCase,
+	#name : 'PRBuildStrategyTest',
+	#superclass : 'TestCase',
 	#instVars : [
 		'project'
 	],
-	#category : #'Pillar-Tests-ExporterCore-Base'
+	#category : 'Pillar-Tests-ExporterCore-Base',
+	#package : 'Pillar-Tests-ExporterCore',
+	#tag : 'Base'
 }
 
-{ #category : #data }
+{ #category : 'data' }
 PRBuildStrategyTest >> chap1Content [
 	^
 	'
@@ -21,7 +23,7 @@ If you do not know Pillar, check its documentation at *http://github.com/pillar-
 '
 ]
 
-{ #category : #data }
+{ #category : 'data' }
 PRBuildStrategyTest >> chap2Content [
 	^
 '
@@ -35,7 +37,7 @@ Some template are already written, but if you want to have your own, you had two
 '
 ]
 
-{ #category : #data }
+{ #category : 'data' }
 PRBuildStrategyTest >> configContent [
 	^ '{
   "base_url": "",
@@ -49,7 +51,7 @@ PRBuildStrategyTest >> configContent [
 }'
 ]
 
-{ #category : #data }
+{ #category : 'data' }
 PRBuildStrategyTest >> indexContent [
 	^
 	'!My First book
@@ -59,7 +61,7 @@ PRBuildStrategyTest >> indexContent [
 		'
 ]
 
-{ #category : #running }
+{ #category : 'running' }
 PRBuildStrategyTest >> setUp [
 	"Create a setup project with: index.pillar, a conf, and Chapters with Chap1 and Chap2"
 
@@ -74,7 +76,7 @@ PRBuildStrategyTest >> setUp [
 	project := PRProject new baseDirectory: fileSystem
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRBuildStrategyTest >> testPRBuildAllStrategy [
 	| f fsorted|
 	f := PRBuildAllStrategy new filesToBuildOn: project.
@@ -92,7 +94,7 @@ PRBuildStrategyTest >> testPRBuildAllStrategy [
 	self assert: f fourth file fullName equals: '/index.pillar'
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRBuildStrategyTest >> testPRBuildListStrategy [
 	| f |
 	f := PRBuildListStrategy new
@@ -103,7 +105,7 @@ PRBuildStrategyTest >> testPRBuildListStrategy [
 							includesAll: #('/index.pillar' '/chapter1.pillar')).
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRBuildStrategyTest >> testPRBuildListStrategyDoubleEntry [
 	| f |
 	f := PRBuildListStrategy new
@@ -113,7 +115,7 @@ PRBuildStrategyTest >> testPRBuildListStrategyDoubleEntry [
 	self assert: f anyOne file fullName equals: '/chapter1.pillar'
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRBuildStrategyTest >> testPRBuildRootMainStrategy [
 	| f |
 	f := PRBuildRootMainStrategy new filesToBuildOn: project.

diff --git a/src/Pillar-Tests-ExporterCore/PRCascadingCounterTest.class.st b/src/Pillar-Tests-ExporterCore/PRCascadingCounterTest.class.st
@@ -2,17 +2,19 @@
 I test the PRCascadingCounter class.
 "
 Class {
-	#name : #PRCascadingCounterTest,
-	#superclass : #TestCase,
-	#category : #'Pillar-Tests-ExporterCore-Various'
+	#name : 'PRCascadingCounterTest',
+	#superclass : 'TestCase',
+	#category : 'Pillar-Tests-ExporterCore-Various',
+	#package : 'Pillar-Tests-ExporterCore',
+	#tag : 'Various'
 }
 
-{ #category : #accessing }
+{ #category : 'accessing' }
 PRCascadingCounterTest >> packageNamesUnderTest [
 	^ #('Pillar-ExporterCore')
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRCascadingCounterTest >> testElementsForNonSimpleCounter [
 	| counter |
 	counter := PRCascadingCounter maxLevel: 5.
@@ -21,7 +23,7 @@ PRCascadingCounterTest >> testElementsForNonSimpleCounter [
 	self assert: counter elements equals: { 0 . 1 }
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRCascadingCounterTest >> testElementsForSimpleCounter [
 	| counter |
 	counter := PRCascadingCounter maxLevel: 5.
@@ -30,7 +32,7 @@ PRCascadingCounterTest >> testElementsForSimpleCounter [
 	self assert: counter elements equals: { 1 }
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRCascadingCounterTest >> testIsValidCounter [
 	self assert: (PRCascadingCounter maxLevel: 5) isValidCounter.
 	self deny: (PRNotFoundCounter maxLevel: 5) isValidCounter

diff --git a/src/Pillar-Tests-ExporterCore/PRCitationTransformerTest.class.st b/src/Pillar-Tests-ExporterCore/PRCitationTransformerTest.class.st
@@ -1,15 +1,17 @@
 Class {
-	#name : #PRCitationTransformerTest,
-	#superclass : #PRNodeTransformerTest,
-	#category : 'Pillar-Tests-ExporterCore-Transformers'
+	#name : 'PRCitationTransformerTest',
+	#superclass : 'PRNodeTransformerTest',
+	#category : 'Pillar-Tests-ExporterCore-Transformers',
+	#package : 'Pillar-Tests-ExporterCore',
+	#tag : 'Transformers'
 }
 
-{ #category : #private }
+{ #category : 'private' }
 PRCitationTransformerTest >> actualClass [
 	^PRCitationTransformer
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRCitationTransformerTest >> testTransformWithAnnotationAndText [
 	| input expected |
 	input := PRDocument new
@@ -29,7 +31,7 @@ PRCitationTransformerTest >> testTransformWithAnnotationAndText [
 	self assert: (self executePhase: input with: self configuration) equals: expected
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRCitationTransformerTest >> testTransformWithListInList [
 	| input expected |
 	input := PRDocument new
@@ -51,7 +53,7 @@ PRCitationTransformerTest >> testTransformWithListInList [
 	self assert: (self executePhase: input with: self configuration) equals: expected
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRCitationTransformerTest >> testTransformWithOneAnnotation [
 	| input expected |
 	input := PRDocument new
@@ -61,7 +63,7 @@ PRCitationTransformerTest >> testTransformWithOneAnnotation [
 	self assert: (self executePhase: input with: self configuration) equals: expected
 ]
 
-{ #category : #tests }
+{ #category : 'tests' }
 PRCitationTransformerTest >> testTransformWithOneAnnotationithRef [
 	| input expected |
 	input := PRDocument new