Skip to content

Commit 83ef0fb

Browse files
authored
Merge pull request #251 from nr-linbaker/jboss_ver
fix: Update compatible version check for Jboss EAP
2 parents 241a002 + c2484c6 commit 83ef0fb

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

tasks/java/appserver/jbosseap.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ func (p JavaAppserverJbossEapCheck) getJbossFileLinux() (string, error) {
224224
}
225225

226226
func versionCheckJboss(version string) (status tasks.Status, summary string) {
227-
compatibilityRequirements := []string{"6-6.*"}
227+
compatibilityRequirements := []string{"6+"}
228228
versionSplit := strings.Split(version, ".")
229229
var err error
230230
isCompatible := false

tasks/java/appserver/jbosseap_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,12 @@ func TestVersionCheckJboss(t *testing.T) {
4343
version string
4444
want tasks.Status
4545
}{
46-
{version: "7.0", want: tasks.Failure},
46+
{version: "8.0", want: tasks.Success},
4747
{version: "6.2", want: tasks.Success},
4848
{version: "5.2", want: tasks.Failure},
4949
{version: "2.11", want: tasks.Failure},
5050
{version: "not valid version", want: tasks.Error},
51-
{version: "7.11.2.GA", want: tasks.Failure},
51+
{version: "5.11.2.GA", want: tasks.Failure},
5252
{version: "6.1.2.GA", want: tasks.Success},
5353
}
5454

0 commit comments

Comments
 (0)