-
Notifications
You must be signed in to change notification settings - Fork 328
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
[SKIP CI] add test for cli #100
Conversation
|
||
jest.mock('ejs', () => { | ||
const ejsMock = jest.createMockFromModule('ejs') as any; | ||
ejsMock.renderFile = jest.fn(); |
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.
This line seems not necessary, it has been done by createMockFromModule
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.
removed
|
||
// 4. Save the rendered schema | ||
function makeSchema(className: string, data: string): void { | ||
export async function makeSchema(className: string, data: string): Promise<void> { |
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.
this abstraction doesn't make sense to me, it is simply wrap of fs.write and once be called once
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.
merged into renderTemplate
* Move enum under schema (#1527) * need fix query get enum * fix enum query, clean all enums * add logic to handle original enum type * fix check exclude constrain check missing schema and table (#1537) * [SKIP CI] Prerelease * manual prerelease node for node-core update (#1538) * [SKIP CI] Prerelease * resolution for sequelize and prerelease node (#1539) * [SKIP CI] Prerelease * fixed the version of sequelize and prerelease node (#1540) * [SKIP CI] Prerelease * improve force clean with deprecated enum (#1541) * [SKIP CI] Prerelease * Remove exclude constraint (#1543) * deprecate exclude contraint * remove unused * [SKIP CI] Prerelease * [release] 20230306 (#1544) * [release] 20230307 (#1545) * bump yarn.lock, update imports (#1546) * [SKIP CI] Prerelease * [release] 20230307 (#1547) * increase wsProvider timeout (#1550) * increase wsProvider timeout * remove .mmr dir * [SKIP CI] Prerelease * Use performance.now instead of date for profiler. (#1549) * add SmartBatchService and BlockSizeBuffer (#1506) * add SmartBatchService and BlockSizeBuffer * check memory usage before fetching a batch * use v8 heap usage statistics * set block to null after indexing * extend smart batch service to workers * make smart batch size injectible * rebase * bug fixes * code cleanup * assign workers based on free memory * fix liniting * cleanup code * cleanup logs * clear blocksizebuffer inside smartbatchservice --------- Co-authored-by: Scott Twiname <skott.twiname@gmail.com> * Fix build issue with profiler * [SKIP CI] Prerelease * fix abi codegen runtime validation (#1552) * [SKIP CI] Prerelease * [release] 20230309 Cli (#1553) * Fix enum under schema not being escaped (#1555) * [SKIP CI] Prerelease --------- Co-authored-by: Jay Ji <jiqiang90@hotmail.com> Co-authored-by: Ben <89335033+bz888@users.noreply.github.com> Co-authored-by: Scott Twiname <skott.twiname@gmail.com>
No description provided.