Conversation
β add support for nocache ππ add missing str includes any of util π convert to typescript
β¨ add noApi to include
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All seems legit, Let's just move the tests to Fluent.test.ts
src/tests/Arithmetic.test.ts
Outdated
"isArithmetic should detect arithmetics properly"() { | ||
const isArithmeticTrue = Fluent.isArithmetic('!^>[index.js] +[**/*.js] -path') | ||
const isArithmeticFalse = Fluent.isArithmetic('eh...') | ||
should(isArithmeticTrue).deepEqual(true) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
beTrue()
src/tests/Arithmetic.test.ts
Outdated
.finishBundle() | ||
.finish() | ||
|
||
should(typeof singleBundle).deepEqual('string') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should(singleBundle).beString().equal("actual string should be here")
src/tests/Arithmetic.test.ts
Outdated
.finishBundle() | ||
.finish() | ||
|
||
should(typeof multipleBundles).deepEqual('object') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
beObject().deepEqual(/* would be cool to have results here */)
src/tests/Arithmetic.test.ts
Outdated
let parsed = Arithmetic.parse(singleBundle); | ||
|
||
should(parsed.str.includes('!')) | ||
.deepEqual(true) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
beTrue()
src/tests/Arithmetic.test.ts
Outdated
|
||
const singleBundle = result.finish() | ||
|
||
should(typeof singleBundle).deepEqual('string') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
beString().equal("actual string")
src/arithmetic/Fluent.ts
Outdated
public arithmetics: string; | ||
public noDeps: boolean; | ||
|
||
constructor(name, arithmetics) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
public arithmetics : string
src/arithmetic/Fluent.ts
Outdated
this.cmds = [] | ||
this.str = `` | ||
this.name = name | ||
this.arithmetics = arithmetics |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
goes to contructor
πΎ shorter ts syntax for constructors
βπ¬ add more thorough tests following the standard testing formula π¦ export Fluent in index
#336
π remove old implementation in index
πΎ simplify
ππ add missing str includes any of util
π convert to typescript
π¬ tests for all
π¦ import export Fluent in Arithmetics
β add support for nocache
β check for ^ & ~ in isArithmetic