Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

definedTests on a multi project build.sbt displays differently than flat build.sbt file #7339

Open
er1c opened this issue Jul 25, 2023 · 1 comment
Labels

Comments

@er1c
Copy link
Contributor

er1c commented Jul 25, 2023

steps

sbt Test/definedTests

Flat build.sbt

[info] * Test integration.LeadFunnelForSupplySpec : subclass(false, org.scalatest.Suite)
[info] * Test integration.SavingsReportingSpec : subclass(false, org.scalatest.Suite)
[info] * Test services.LeadFunnelAggregationServiceSpec : subclass(false, org.scalatest.Suite)
[info] * Test services.LeadFunnelCalculationServiceSpec : subclass(false, org.scalatest.Suite)
[info] * Test daos.LeadFunnelsDaoSpec : subclass(false, org.scalatest.Suite)
[info] * Test integration.LoanReportingSpec : subclass(false, org.scalatest.Suite)
[info] * Test integration.LifeInsuranceReportingSpec : subclass(false, org.scalatest.Suite)
[info] * Test services.DetectNewSubAccountFinancialLineItemsServiceSpec : subclass(false, org.scalatest.Suite)
[info] * Test daos.SubAccountFinancialLineItemDaoPropertySpec : subclass(false, org.scalacheck.Properties)
[info] * Test services.StaleLeadServiceSpec : subclass(false, org.scalatest.Suite)
[info] * Test integration.LeadFunnelRecalculationSpec : subclass(false, org.scalatest.Suite)
[info] * Test integration.CreditCardReportingSpec : subclass(false, org.scalatest.Suite)
[info] * Test daos.ReplicationHubDaoSpec : subclass(false, org.scalatest.Suite)
[info] * Test daos.SubAccountFinancialLineItemDaoSpec : subclass(false, org.scalatest.Suite)
[success] Total time: 1 s, completed Jul 25, 2023, 4:33:23 PM

Multi-project

[info] libTest / Test / definedTests
[info] 	Vector()
[info] svc / Test / definedTests
[info] 	Vector(Test integration.UserDeactivationsSpec : subclass(false, org.scalatest.Suite), Test integration.middleware.BearerTokenLookupSpec : subclass(false, org.scalatest.Suite), Test integration.RoleScopesSpec : subclass(false, org.scalatest.Suite), Test integration.AccountsSpec : subclass(false, org.scalatest.Suite), Test services.UserDeactivationServiceSpec : subclass(false, org.scalatest.Suite), Test services.UserServiceSpec : subclass(false, org.scalatest.Suite), Test services.daos.AccessTokenAuthDaoSpec : subclass(false, org.scalatest.Suite), Test integration.ApplicationsSpec : subclass(false, org.scalatest.Suite), Test services.CompanyServiceSpec : subclass(false, org.scalatest.Suite), Test integration.UsersSpec : subclass(false, org.scalatest.Suite), Test integration.CompaniesSpec : subclass(false, org.scalatest.Suite), Test services.ResetPasswordServiceTest : subclass(false, org.scalatest.Suite), Test services.AccountServiceSpec : subclass(false, org.scalatest.Suite), Test integration.internal.v2.AuthSpec : subclass(false, org.scalatest.Suite), Test integration.RichAccountsSpec : subclass(false, org.scalatest.Suite), Test messaging.SupplySubAccountStateChangedConsumerSpec : subclass(false, org.scalatest.Suite), Test caches.AccountCacheSpec : subclass(false, org.scalatest.Suite), Test messaging.AccountStateChangedConsumerSpec : subclass(false, org.scalatest.Suite), Test services.AuthServiceSpec : subclass(false, org.scalatest.Suite), Test integration.LoginsSpec : subclass(false, org.scalatest.Suite), Test integration.internal.v1.AuthSpec : subclass(false, org.scalatest.Suite), Test integration.ThirdPartyCredentialsSpec : subclass(false, org.scalatest.Suite), Test services.ScopeServiceSpec : subclass(false, org.scalatest.Suite), Test integration.RoleSpec : subclass(false, org.scalatest.Suite), Test middleware.AuthSpec : subclass(false, org.scalatest.Suite), Test integration.RichAccessTokensSpec : subclass(false, org.scalatest.Suite), Test integration.ServicesSpec : subclass(false, org.scalatest.Suite), Test services.daos.OktaUsersDaoSpec : subclass(false, org.scalatest.Suite), Test services.daos.AccessTokenDaoSpec : subclass(false, org.scalatest.Suite), Test integration.PasswordsSpec : subclass(false, org.scalatest.Suite), Test services.v2.AuthServiceSpec : subclass(false, org.scalatest.Suite), Test integration.ScopesSpec : subclass(false, org.scalatest.Suite), Test lib.CompanyResourceOwnerCacheSpec : subclass(false, org.scalatest.Suite), Test integration.client.AccountClientSpec : subclass(false, org.scalatest.Suite), Test integration.CompanyClientSpec : subclass(false, org.scalatest.Suite), Test integration.CorSpec : subclass(false, org.scalatest.Suite), Test integration.AccessTokensSpec : subclass(false, org.scalatest.Suite), Test services.v2.AuthorizationServiceSpec : subclass(false, org.scalatest.Suite), Test messaging.UserStateChangedConsumerSpec : subclass(false, org.scalatest.Suite), Test integration.AuthsSpec : subclass(false, org.scalatest.Suite))
[info] oktaUserSync / Test / definedTests

problem

I would expect the output to be shown the same way

expectation

notes

I'll create a more minimalistic example, creating so I can keep track of this.

@er1c er1c added the Bug label Jul 25, 2023
@eed3si9n
Copy link
Member

@er1c Thanks for the report. I don't know if I'd call it a bug, but it does seem like a good idea to unify to the markdown list on multi-project.

Likely relevant part of the code is

def printSettings(xs: Seq[KeyValue[_]], print: String => Unit)(
implicit display: Show[ScopedKey[_]]
): Unit =
xs match {
case KeyValue(_, x: Seq[_]) :: Nil => print(x.mkString("* ", "\n* ", ""))
case KeyValue(_, x) :: Nil => print(x.toString)
case _ =>
xs foreach (kv => print(display.show(kv.key) + "\n\t" + kv.value.toString))
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants