Skip to content

test example#14

Closed
see-sharp-dev wants to merge 2 commits intoapache:masterfrom
see-sharp-dev:failing-test
Closed

test example#14
see-sharp-dev wants to merge 2 commits intoapache:masterfrom
see-sharp-dev:failing-test

Conversation

@see-sharp-dev
Copy link
Copy Markdown

This is an example of failing tests.
If I ran go test ./... -race the data race will be detected.

WARNING: DATA RACE
Read at 0x00c0000160f8 by goroutine 6:
  runtime.slicecopy()
      /usr/local/go/src/runtime/slice.go:355 +0x0
  github.com/casbin/govaluate.parseTokens()
      /xxx/tmp/govaluate/parsing.go:54 +0x3cc
  github.com/casbin/govaluate.NewEvaluableExpressionWithFunctions()
      /xxx/tmp/govaluate/EvaluableExpression.go:100 +0xb4
  github.com/casbin/govaluate.NewEvaluableExpression()
      /xxx/tmp/govaluate/EvaluableExpression.go:48 +0x94
  github.com/casbin/govaluate.TestA()
      /xxx/tmp/govaluate/EvaluableExpression_test.go:8 +0x34
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:1690 +0x184
  testing.(*T).Run.gowrap1()
      /usr/local/go/src/testing/testing.go:1743 +0x40

Previous write at 0x00c0000160f8 by goroutine 7:
  runtime.slicecopy()
      /usr/local/go/src/runtime/slice.go:355 +0x0
  github.com/casbin/govaluate.parseTokens()
      /xxx/tmp/govaluate/parsing.go:54 +0x3cc
  github.com/casbin/govaluate.NewEvaluableExpressionWithFunctions()
      /xxx/tmp/govaluate/EvaluableExpression.go:100 +0xb4
  github.com/casbin/govaluate.NewEvaluableExpression()
      /xxx/tmp/govaluate/EvaluableExpression.go:48 +0x94
  github.com/casbin/govaluate.TestB()
      /xxx/tmp/govaluate/EvaluableExpression_test.go:18 +0x34
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:1690 +0x184
  testing.(*T).Run.gowrap1()
      /usr/local/go/src/testing/testing.go:1743 +0x40

Goroutine 6 (running) created at:
  testing.(*T).Run()
      /usr/local/go/src/testing/testing.go:1743 +0x5e0
  testing.runTests.func1()
      /usr/local/go/src/testing/testing.go:2168 +0x80
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:1690 +0x184
  testing.runTests()
      /usr/local/go/src/testing/testing.go:2166 +0x6e0
  testing.(*M).Run()
      /usr/local/go/src/testing/testing.go:2034 +0xb74
  main.main()
      _testmain.go:151 +0x110

Goroutine 7 (running) created at:
  testing.(*T).Run()
      /usr/local/go/src/testing/testing.go:1743 +0x5e0
  testing.runTests.func1()
      /usr/local/go/src/testing/testing.go:2168 +0x80
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:1690 +0x184
  testing.runTests()
      /usr/local/go/src/testing/testing.go:2166 +0x6e0
  testing.(*M).Run()
      /usr/local/go/src/testing/testing.go:2034 +0xb74
  main.main()
      _testmain.go:151 +0x110

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 13, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


see-sharp-dev seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@see-sharp-dev see-sharp-dev deleted the failing-test branch May 13, 2025 16:34
@see-sharp-dev see-sharp-dev restored the failing-test branch May 13, 2025 16:35
@hsluoyz hsluoyz reopened this May 13, 2025
@hsluoyz
Copy link
Copy Markdown
Member

hsluoyz commented May 13, 2025

@see-sharp-dev why close?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants