allow for TimeSeries with just timestamps specified

[bendichter]

Motivation

Previously, if you wanted to use mock_TimeSeries while specifying timestamps, you would also need to set rate=None.

from pynwb.testing.mock.base import mock_TimeSeries

mock_TimeSeries(timestamps=[1,2,3,4])  # this used to fail

You had to specify rate=None explicitly, which is not ideal

mock_TimeSeries(timestamps=[1,2,3,4], rate=None)  # this was the fix

Now, you don't need to.

from pynwb.testing.mock.base import mock_TimeSeries

mock_TimeSeries()  # this works
mock_TimeSeries(timestamps=[1,2,3,4])  # this works
mock_TimeSeries(rate=10.0)  # this works
mock_TimeSeries(timestamps=[1,2,3,4], rate=10.0)  # this rightfully fails

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7d19192) 91.95% compared to head (d3e2dd8) 91.95%.

Additional details and impacted files

@@           Coverage Diff           @@
##              dev    #1848   +/-   ##
=======================================
  Coverage   91.95%   91.95%           
=======================================
  Files          27       27           
  Lines        2661     2661           
  Branches      699      699           
=======================================
  Hits         2447     2447           
  Misses        142      142           
  Partials       72       72           

Flag	Coverage Δ
integration	70.80% <ø> (ø)
unit	84.14% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.