Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#105: Report: Add date shortcut last_month #158

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
62 changes: 62 additions & 0 deletions spec/commands/report.date.shortcuts.spec.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
const Sinon = require('sinon');
const program = require('commander');
const { assert } = require('chai');
const Config = require('../../src/include/file-config');
const moment = require('moment');
const report = require('../../src/models/report');

describe('Command Report. Date shortcuts', () => {
it('Shortcut last_month registered', () => {
/** @type {Sinon.SinonMock} */
const stub = Sinon.mock(program);

/** @type {Sinon.SinonSpy} */
const spy = Sinon.spy(program, 'option');

stub.expects('parse').throws('Test exception to prevent further execution of gtt-report')
try {
require('../../src/gtt-report');
} catch (e) {}

try {
assert(spy.calledWith('--last_month'), 'last_month not registered');
} finally {
spy.restore();
stub.restore();
}
});

it('Shortcut last_month. Sets From and To to Config', () => {
program.last_month = true;
program.args = [];
let configSpy = Sinon.spy(Config.prototype, 'set');

const reportOriginal = Object.getPrototypeOf(report);
let reportStub = Sinon.stub().callsFake(() => {
throw new Error('Test exception to prevent further execution of gtt-report')
});
Object.setPrototypeOf(report, reportStub);

stub = Sinon.mock(program);
stub.expects('parse').returnsThis();

try {
require('../../src/gtt-report');
} catch (e) {}

try {
assert(
configSpy.calledWith('from', moment().subtract(1, 'months').startOf('month')),
'From is not set to Config'
);
assert(
configSpy.calledWith('to', moment().subtract(1, 'months').endOf('month')),
'To is not set to Config'
);
} finally {
configSpy.restore();
Object.setPrototypeOf(report, reportOriginal);
delete require.cache[require.resolve('../../src/models/report')];
}
})
})
5 changes: 5 additions & 0 deletions src/gtt-report.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ program
.option('--today', 'ignores --from and --to and queries entries for today')
.option('--this_week', 'ignores --from and --to and queries entries for this week')
.option('--this_month', 'ignores --from and --to and queries entries for this month')
.option('--last_month', 'ignores --from and --to and queries entries for last month')
.option('-c --closed', 'include closed issues')
.option('-m --milestone <milestone>', 'include issues from the given milestone')
.option('--hours_per_day <hours>', 'hours per day for human readable time formats')
Expand Down Expand Up @@ -138,6 +139,10 @@ if (program.this_month)
config
.set('from', moment().startOf('month'))
.set('to', moment().endOf('month'));
if (program.last_month)
config
.set('from', moment().subtract(1, 'months').startOf('month'))
.set('to', moment().subtract(1, 'months').endOf('month'));

Cli.quiet = config.get('quiet');
Cli.verbose = config.get('_verbose');
Expand Down