Added CLI Flags for Package name customization for Model, Table, View and Enum

[jamius19]

Goal

Jet doesn't currently support custom package names for the generated files for Models, Tables, Views, and Enums. It always uses the fixed model, table, view, and enum names for these packages. This PR will allow users to pass OPTIONAL CLI arguments to Jet for custom package names.

Features Added

Users can customize the package names through these flags,

1. -model-pkg for the model package name
2. -table-pkg for the table package name
3. -view-pkg for the view package name
4. -enum-pkg for the enum package name

If any or all of these flags are absent, the default package names (e.g., model for Model) will be used making this a NON Breaking change.

I've also added a corresponding integration test for these optional CLI arguments in Postgres. This should be sufficient as all databases use the main.go:genTemplate() function that was changed to accommodate this feature.

For additional context, please see Issue #413

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.25%. Comparing base (3ca1776) to head (71ae4ed).
Report is 9 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #424   +/-   ##
=======================================
  Coverage   91.25%   91.25%           
=======================================
  Files         134      134           
  Lines        7755     7755           
=======================================
  Hits         7077     7077           
  Misses        509      509           
  Partials      169      169           

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

[jamius19]

Rebased my branch with master

[jamius19]

@go-jet Please don't merge this PR yet, relative directory names starting with ./ or .\ and nested directory names don't work on Windows. Most probably related to Issue #426

I'll investigate and update here.

[jamius19]

Issue #426 is related to how PowerShell handles arguments.
TL;DR Arguments should be passed as quoted strings in PowerShell.

However, if a quoted nested directory is passed as the value of -model-pkg, it breaks on Windows.
I'm checking up on it now.

[jamius19]

Fixed path compatibility issues on Windows.

[jamius19]

Since we're passing a path for the model package, wouldn't it be better if the flag was model-pkg-dir?

Same goes for the other three as well.

[go-jet]

Since we're passing a path for the model package, wouldn't it be better if the flag was model-pkg-dir?

Same goes for the other three as well.

Shouldn't be path then (model-pkg-path)? Since directory is technically just a container.
Maybe we can also drop pkg(it kind of implies), and add rel. - rel-model-path, rel-table-path, rel-enum-path, rel-view-path

[jamius19]

I agree with both points that it should be path and that the pkg is redundant.
I also think the addition of rel in these new flags will help indicate that they are relative to something (the path flag in this case).

I'm changing the flags like this -rel-model-path then.

[go-jet]

You'll need to update this names as well.

[jamius19]

Yeah, my bad.
Pushed the changes already.

[go-jet]

Great. LGTM. 👍