fix: change id to indentifier for software

[satra]

an RRID can be associated with different versions, therefore cannot be used as the id of an object. this fixes this in a test.

[codecov]

Codecov Report

Merging #46 (4b34f36) into master (3608577) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #46   +/-   ##
=======================================
  Coverage   95.87%   95.87%           
=======================================
  Files          11       11           
  Lines        1042     1042           
=======================================
  Hits          999      999           
  Misses         43       43           

Flag	Coverage Δ
unittests	95.87% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
dandischema/tests/test_datacite.py	100.00% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3608577...4b34f36. Read the comment docs.

[waxlamp]

@satra, who is able to review this change? Could you nominate someone in the sidebar?

[satra]

@waxlamp - i added @dchiquito just because this is also related to the dandi/dandi-archive#364 PR.

this is a rather innocuous change for this repo. so not releasing a new version at this point.

[yarikoptic]

@satra we still need this PR to be merged right? (hard to assess the necessity since tests seemed to be ok without it and with it here ;-)) your PR in dandi-api was superseded by dandi/dandi-archive#372 (merged already)

[satra]

yes, this is more of a clarity PR than a test PR. i just merged with master. we can merge when tests pass.