Skip to content

Commit

Permalink
Merge branch 'gemstone2.4_dev' into gemstone2.4 :: final fix for Issue
Browse files Browse the repository at this point in the history
…#136

Conflicts:
	repository/MonticelloFileTree-Tests.package/MCFileTreeIssue136Test.class/instance/expectedFailures.st
	repository/MonticelloFileTree-Tests.package/MCFileTreeIssue136Test.class/instance/tearDownPackagesList.st

Conflicts:

	repository/MonticelloFileTree-Tests.package/MCFileTreeIssue97Test.class/instance/testLoad.st
  • Loading branch information
dalehenrich committed Feb 1, 2015
1 parent a108428 commit 21a1093
Show file tree
Hide file tree
Showing 25 changed files with 99 additions and 20 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,5 @@ validateExtensionMethodCategory: categoryName for: className selector: selector
self
error:
'Method protocol ' , categoryName printString , ' for the method '
, selector printString , ' in class ' , className printString
, selector asString printString , ' in class ' , className printString
, ' is inconsistent with the package name ' , prefix printString
Original file line number Diff line number Diff line change
@@ -1,15 +1,21 @@
validation
verifyCategory: categoryName matches: prefix
verifyCategory: categoryName matches: basicPackageName
"https://github.com/dalehenrich/filetree/issues/136"

"copied from PackageInfo>>category:matches:"
"copied from PackageInfo>>category:matches: and GoferVersionReference>>parseName:"

| prefixSize catSize |
| prefixSize catSize packagePrefix |
categoryName ifNil: [ ^ false ].
packagePrefix := basicPackageName.
(packagePrefix includes: $.)
ifTrue: [
"exclude branch name"
packagePrefix := packagePrefix copyUpTo: $. ].
catSize := categoryName size.
prefixSize := prefix size.
prefixSize := packagePrefix size.
catSize < prefixSize
ifTrue: [ ^ false ].
(categoryName findString: prefix startingAt: 1 caseSensitive: false) = 1
(categoryName findString: packagePrefix startingAt: 1 caseSensitive: false)
= 1
ifFalse: [ ^ false ].
^ (categoryName at: prefix size + 1 ifAbsent: [ ^ true ]) = $-
^ (categoryName at: packagePrefix size + 1 ifAbsent: [ ^ true ]) = $-
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,4 @@
"packageNameFromPackageDirectory" : "dkh 8/10/2012 14:55",
"validateClassCategory:for:" : "dkh 01/31/2015 12:20",
"validateExtensionMethodCategory:for:selector:" : "dkh 01/31/2015 13:41",
"verifyCategory:matches:" : "dkh 01/31/2015 07:05" } }
"verifyCategory:matches:" : "dkh 01/31/2015 15:14" } }
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
(name 'MonticelloFileTree-Core-dkh.178' message 'merged by GitFileTree-MergeDriver' id 'cdbd1539-c54e-4ca1-aead-8a2a29e5a663' date '31 January 2015' time '2:40:28.441134 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.177' message 'Issue #136: cherry pick code from gemstone2.4 branch' id '724c52b7-49cd-4643-92c5-df38aa01040a' date '31 January 2015' time '1:44:33 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.175' message 'Issue #136: cherrypick the pharo1.1_dev changes' id 'c302b9e1-7356-4dc3-8e86-4ebfc5343cef' date '31 January 2015' time '12:45:02 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.174' message 'merged by GitFileTree-MergeDriver' id 'b508a3f8-7639-4940-9764-b5b6001beac0' date '31 January 2015' time '12:32:29 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.172' message 'Issue #136: back out the class category validation, due to faulty validation logic (https://github.com/GsDevKit/gsDevKitHome/issues/53)' id 'e9bf1ab7-9a08-4581-85b8-d7c076e705fc' date '31 January 2015' time '12:31:38 pm' author 'dkh' ancestors () stepChildren ())(name 'MonticelloFileTree-Core-dkh.173' message 'merged by GitFileTree-MergeDriver' id '777f204b-8a94-493a-b198-922b83da74be' date '31 January 2015' time '11:49:40 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.171' message 'merged by GitFileTree-MergeDriver' id '3521c0b7-456c-4304-b3dc-97ba5ab82a02' date '31 January 2015' time '11:05:45 am' author 'dkh' ancestors () stepChildren ())(name 'MonticelloFileTree-Core-dkh.172' message 'merged by GitFileTree-MergeDriver' id '9791aa8a-58ce-46e1-9f4a-749eb74b56a8' date '31 January 2015' time '11:42:10 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.171' message 'merged by GitFileTree-MergeDriver' id 'f68da1e3-a57c-4a24-95c2-25db920bdbf2' date '31 January 2015' time '11:09:36 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.158' message 'ensure ss3 version is in ancestry for release: ''1.0.6''' id '592e6387-8ece-4b37-94da-204fb1a93f55' date '8 September 2013' time '7:16:27 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core.pharo11-dkh.157' message 'release 1.0.5:
(name 'MonticelloFileTree-Core-dkh.182' message 'merged by GitFileTree-MergeDriver' id 'a62b8a94-e8f6-4c19-ad79-d1a5e2bb2e05' date '31 January 2015' time '7:27:08.228229 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.181' message 'merged by GitFileTree-MergeDriver' id '99a3ccf1-2fa4-4405-b015-9b104b284849' date '31 January 2015' time '7:21:07.705222 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.176' message 'Issue #136: method validation is incorrect as well (DUH) back it out for https://github.com/GsDevKit/gsDevKitHome/issues/53: ' id '382dd5af-be9d-460c-86d3-7fd92f2fa90e' date '31 January 2015' time '12:50:01 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.174' message 'merged by GitFileTree-MergeDriver' id 'b508a3f8-7639-4940-9764-b5b6001beac0' date '31 January 2015' time '12:32:29 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.172' message 'Issue #136: back out the class category validation, due to faulty validation logic (https://github.com/GsDevKit/gsDevKitHome/issues/53)' id 'e9bf1ab7-9a08-4581-85b8-d7c076e705fc' date '31 January 2015' time '12:31:38 pm' author 'dkh' ancestors () stepChildren ())(name 'MonticelloFileTree-Core-dkh.173' message 'merged by GitFileTree-MergeDriver' id '777f204b-8a94-493a-b198-922b83da74be' date '31 January 2015' time '11:49:40 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.171' message 'merged by GitFileTree-MergeDriver' id '3521c0b7-456c-4304-b3dc-97ba5ab82a02' date '31 January 2015' time '11:05:45 am' author 'dkh' ancestors () stepChildren ())(name 'MonticelloFileTree-Core-dkh.172' message 'merged by GitFileTree-MergeDriver' id '9791aa8a-58ce-46e1-9f4a-749eb74b56a8' date '31 January 2015' time '11:42:10 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.171' message 'merged by GitFileTree-MergeDriver' id 'f68da1e3-a57c-4a24-95c2-25db920bdbf2' date '31 January 2015' time '11:09:36 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.158' message 'ensure ss3 version is in ancestry for release: ''1.0.6''' id '592e6387-8ece-4b37-94da-204fb1a93f55' date '8 September 2013' time '7:16:27 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core.pharo11-dkh.157' message 'release 1.0.5:

-fix Issue #11: Traits and Script support
- fix Issue #92: Mac case insensitivity `must` be addressed
Expand Down Expand Up @@ -115,4 +115,4 @@ for details: https://github.com/dalehenrich/filetree/issues?direction=desc&miles
- fix Issue #92: Mac case insensitivity `must` be addressed
- fix Issue #97: Load packages from repos with no monticello meta data
- fix Issue #101: Refactor MCFileTreeStCypressWriter>>writeDefinitions: to ease Issue #11 integration
for details: https://github.com/dalehenrich/filetree/issues?direction=desc&milestone=9&page=1&sort=updated&state=closed ' id '6c81f18a-d5f2-4238-9334-d3d58a290617' date '11 August 2013' time '7:58:43 am' author 'dkh' ancestors () stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())
for details: https://github.com/dalehenrich/filetree/issues?direction=desc&milestone=9&page=1&sort=updated&state=closed ' id '6c81f18a-d5f2-4238-9334-d3d58a290617' date '11 August 2013' time '7:58:43 am' author 'dkh' ancestors () stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())(name 'MonticelloFileTree-Core-dkh.180' message 'Issue #136: fix for issue causing https://github.com/GsDevKit/gsDevKitHome/issues/53 and add tests' id 'f5423489-829b-4212-b901-e50951d806c7' date '31 January 2015' time '3:43:11 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.179' message 'merged by GitFileTree-MergeDriver' id '15c8cf95-5444-4a1d-8b1c-9d6bec06190e' date '31 January 2015' time '2:48:52 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.178' message 'merged by GitFileTree-MergeDriver' id 'cdbd1539-c54e-4ca1-aead-8a2a29e5a663' date '31 January 2015' time '2:40:28 am' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.177' message 'Issue #136: cherry pick code from gemstone2.4 branch' id '724c52b7-49cd-4643-92c5-df38aa01040a' date '31 January 2015' time '1:44:33 pm' author 'dkh' ancestors ((name 'MonticelloFileTree-Core-dkh.175' message 'Issue #136: cherrypick the pharo1.1_dev changes' id 'c302b9e1-7356-4dc3-8e86-4ebfc5343cef' date '31 January 2015' time '12:45:02 pm' author 'dkh' ancestors ((id 'b508a3f8-7639-4940-9764-b5b6001beac0')) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())) stepChildren ())

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
running
tearDownPackagesList
^ #('Issue136-1' 'Issue136-2' 'Issue97')
^ #('Issue136-1' 'Issue136-2' 'Issue97' 'Issue136-1.v3' 'Issue136-2.v3')
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,8 @@ testClassError
hitError := (description := ex description)
endsWith:
'Class category name ''Issue97'' for the class ''CypressMockIssue97'' is inconsistent with the package name ''Issue136-1''' ].
Transcript
cr;
show:
'testClassError: ' , hitError printString , ' :: ' , description printString.
self assert: hitError description: description
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
tests
testClassV3
| packageName |
packageName := 'Issue136-1.v3'.
{packageName} do: [ :pn | self deny: (self hasPackage: pn) ].
Gofer new
disablePackageCache;
repository: (self getTestRepository: 'issue136');
version: 'Issue136-1.v3-cypress.1';
load.
self deny: (Smalltalk at: #'CypressMockIssue97' ifAbsent: [ ]) == nil
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,9 @@ testExtensionMethodError
hitError := (description := ex description)
endsWith:
'Method protocol ''*Issue97'' for the method ''isCypressMockIssue97'' in class ''Object'' is inconsistent with the package name ''*issue136-2''' ].
Transcript
cr;
show:
'testExtensionMethodError: ' , hitError printString , ' :: '
, description printString.
self assert: hitError description: description
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
tests
testExtensionMethodV3
| packageName |
packageName := 'Issue136-2.v3'.
{packageName} do: [ :pn | self deny: (self hasPackage: pn) ].
Gofer new
disablePackageCache;
repository: (self getTestRepository: 'issue136');
version: 'Issue136-2.v3-cypress.1';
load.
self assert: (Object selectors includes: #'isCypressMockIssue97')
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@
"instance" : {
"expectedFailures" : "dkh 01/31/2015 12:48",
"tearDown" : "dkh 01/31/2015 12:59",
"tearDownPackagesList" : "dkh 01/31/2015 13:28",
"tearDownPackagesList" : "dkh 01/31/2015 14:59",
"testClassError" : "dkh 01/31/2015 12:43",
"testExtensionMethodError" : "dkh 01/31/2015 12:43" } }
"testClassV3" : "dkh 01/31/2015 15:30",
"testExtensionMethodError" : "dkh 01/31/2015 12:43",
"testExtensionMethodV3" : "dkh 01/31/2015 15:30" } }
Loading

0 comments on commit 21a1093

Please sign in to comment.