mitre / mitre/cedar

On Cypress Measure Bundle

Open
#8 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Status: In Progress Type: Bug
Dominant language
Ruby
Stars
8
Forks
5
PR merge metrics
No merged PRs in 30d

Description

The Cypress measure bundle is needed for Cedar to function properly (as documented in the README). There should be steps in the build process (like ecqm's rakefile?) to add the bundle and a clear error messages when the bundle is missing.

If bundle hasn't been installed the following UI error message isn't helpful:

We're sorry, but something went wrong.

Even the log doesn't mention the bundle

NoMethodError (undefined method `id' for nil:NilClass):
  app/controllers/test_executions/steps_controller.rb:11:in `show'

After looking at the code, I finally realized my mistake 😊

This is also relevant to the docker discussion, as one may have to pass in their NLM username and password as environment variables in order to build successfully.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the bundle requirements in README, then inspect the build process and Rakefile, using the ecqm Rakefile linked in the issue for context. Review app/controllers/test_executions/steps_controller.rb and the reported NoMethodError path. Done means the measure bundle is handled during setup and a missing bundle produces a clear UI or log error.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, ruby
Domain
backend, build-system
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.