Skip to content

Set LiveTestCtestRegex to match CtestRegex in ci.yml across the repo, in a consistent format, following the template project. #6224

Open
@ahsonkhan

Description

@ahsonkhan

Tables' ci.yml was the only one that was inconsistent. All other instances within the repo have the LiveTestCtestRegex value match CtestRegex, so we fixed it in #6223, for consistency.

That said, some use wild-card (*), while others have it wrapped in quotes (""), while the rest leave it as plain-text:
https://github.com/search?q=repo%3AAzure%2Fazure-sdk-for-cpp%20LiveTestCtestRegex&type=code

We should consider picking a viable format, and apply it consistently, to avoid confusion.

A strawman proposal is to have it be unquoted, without the wild card, matching what we have in our template and core project:
azure-template
azure-core
azure-security-keyvault
azure-storage

Open question:
Would it make a difference to the set of tests that get matched/run if the regex value ended with a period . or not?

CtestRegex: azure-template

CtestRegex: "azure-security-keyvault.*"
LiveTestCtestRegex: "azure-security-keyvault.*"

CtestRegex: azure-storage
LineCoverageTarget: 81
BranchCoverageTarget: 47
LiveTestCtestRegex: azure-storage

Metadata

Metadata

Assignees

No one assigned

    Labels

    EngSysThis issue is impacting the engineering system.MQThis issue is part of a "milestone of quality" initiative.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions