Skip to content

Published manifest includes test objects #925

@willgriffin

Description

@willgriffin

Description

The smrt-core published manifest (dist/manifest.json) appears to include ~168 test/mock objects. When a consumer project runs db:setup, these test objects pollute the database with unnecessary tables.

Observed: a dashboard combined manifest had 419 objects total, 168 of which were test fixtures from smrt-core. This resulted in 308 tables in a site database.

Uncertainty

This was discovered in a local-linking context. Needs verification with published packages to confirm whether the published npm artifact also includes test objects, or if this is a linking artifact where the build picks up test fixtures.

Expected behavior

Published manifests should only contain production SmrtObject definitions, not test/mock objects.

Files

  • packages/core/dist/manifest.json (check published artifact)

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions