@@ -15,14 +15,18 @@ package e2e
1515
1616import (
1717 "context"
18+ "io/ioutil"
19+ "os"
1820
21+ "github.com/go-git/go-git/v5"
1922 . "github.com/onsi/ginkgo/v2"
2023 . "github.com/onsi/gomega"
2124
2225 "github.com/ossf/scorecard/v4/checker"
2326 "github.com/ossf/scorecard/v4/checks"
2427 "github.com/ossf/scorecard/v4/clients"
2528 "github.com/ossf/scorecard/v4/clients/githubrepo"
29+ "github.com/ossf/scorecard/v4/clients/localdir"
2630 scut "github.com/ossf/scorecard/v4/utests"
2731)
2832
@@ -81,6 +85,47 @@ var _ = Describe("E2E TEST:"+checks.CheckTokenPermissions, func() {
8185 // UPGRADEv2: to remove.
8286 // Old version.
8387
88+ Expect (result .Error ).Should (BeNil ())
89+ Expect (result .Pass ).Should (BeFalse ())
90+ // New version.
91+ Expect (scut .ValidateTestReturn (nil , "dangerous workflow" , & expected , & result , & dl )).Should (BeTrue ())
92+ })
93+ It ("Should return dangerous workflow for local repoClient" , func () {
94+ dl := scut.TestDetailLogger {}
95+
96+ tmpDir , err := ioutil .TempDir ("" , "" )
97+ Expect (err ).Should (BeNil ())
98+ defer os .RemoveAll (tmpDir )
99+
100+ _ , e := git .PlainClone (tmpDir , false , & git.CloneOptions {
101+ URL : "http://github.com/ossf-tests/scorecard-check-dangerous-workflow-e2e" ,
102+ })
103+ Expect (e ).Should (BeNil ())
104+
105+ repo , err := localdir .MakeLocalDirRepo (tmpDir )
106+ Expect (err ).Should (BeNil ())
107+
108+ x := localdir .CreateLocalDirClient (context .Background (), logger )
109+ err = x .InitRepo (repo , clients .HeadSHA )
110+ Expect (err ).Should (BeNil ())
111+
112+ req := checker.CheckRequest {
113+ Ctx : context .Background (),
114+ RepoClient : x ,
115+ Repo : repo ,
116+ Dlogger : & dl ,
117+ }
118+ expected := scut.TestReturn {
119+ Error : nil ,
120+ Score : checker .MinResultScore ,
121+ NumberOfWarn : 2 ,
122+ NumberOfInfo : 0 ,
123+ NumberOfDebug : 0 ,
124+ }
125+ result := checks .DangerousWorkflow (& req )
126+ // UPGRADEv2: to remove.
127+ // Old version.
128+
84129 Expect (result .Error ).Should (BeNil ())
85130 Expect (result .Pass ).Should (BeFalse ())
86131 // New version.
0 commit comments