Browse Source

Improvements after code review

pull/364/head
Thomas 4 years ago
parent
commit
0acba5806e
  1. 17
      apps/api/src/app/portfolio/portfolio.service.spec.ts

17
apps/api/src/app/portfolio/portfolio.service.spec.ts

@ -17,13 +17,10 @@ describe('PortfolioService', () => {
); );
}); });
/** it('Get annualized performance', async () => {
* Source: https://www.investopedia.com/terms/a/annualized-total-return.asp#annualized-return-formula-and-calculation
*/
fit('Get annualized performance', async () => {
expect( expect(
portfolioService.getAnnualizedPerformancePercent({ portfolioService.getAnnualizedPerformancePercent({
daysInMarket: NaN, daysInMarket: NaN, // differenceInDays of date-fns returns NaN for the same day
netPerformancePercent: 0 netPerformancePercent: 0
}) })
).toEqual(0); ).toEqual(0);
@ -35,6 +32,16 @@ describe('PortfolioService', () => {
}) })
).toEqual(0); ).toEqual(0);
expect(
portfolioService.getAnnualizedPerformancePercent({
daysInMarket: 365, // 1 year
netPerformancePercent: 0.05
})
).toBeCloseTo(0.05);
/**
* Source: https://www.investopedia.com/terms/a/annualized-total-return.asp#annualized-return-formula-and-calculation
*/
expect( expect(
portfolioService.getAnnualizedPerformancePercent({ portfolioService.getAnnualizedPerformancePercent({
daysInMarket: 575, daysInMarket: 575,

Loading…
Cancel
Save