Remove default Rails version

This enables `gem install rails` in the Dockerfile to always install the
latest Rails version instead of the default value in `rails-new`.

Author: skipkayhil

src/docker_client.rs

@@ -5,7 +5,7 @@ pub struct DockerClient {}
 impl DockerClient {
     pub fn build_image(
         ruby_version: &str,
-        rails_version: &str,
+        maybe_rails_version: Option<&str>,
         user_id: Option<u32>,
         group_id: Option<u32>,
         rebuild: bool,
@@ -19,7 +19,9 @@ impl DockerClient {
         }
 
         Self::set_build_arg(&mut command, "RUBY_VERSION", ruby_version);
-        Self::set_build_arg(&mut command, "RAILS_VERSION", rails_version);
+        if let Some(rails_version) = maybe_rails_version {
+            Self::set_build_arg(&mut command, "RAILS_VERSION", rails_version);
+        }
 
         if let Some(id) = user_id {
             Self::set_build_arg(&mut command, "USER_ID", &id.to_string())
@@ -30,14 +32,18 @@ impl DockerClient {
 
         command.arg("-t");
 
-        Self::set_image_name(&mut command, ruby_version, rails_version);
+        Self::set_image_name(&mut command, ruby_version, maybe_rails_version);
 
         command.arg("-").stdin(Stdio::piped());
 
         command
     }
 
-    pub fn run_image(ruby_version: &str, rails_version: &str, args: Vec<String>) -> Command {
+    pub fn run_image(
+        ruby_version: &str,
+        rails_version: Option<&str>,
+        args: Vec<String>,
+    ) -> Command {
         let mut command = Self::run();
 
         Self::set_workdir(&mut command);
@@ -47,7 +53,7 @@ impl DockerClient {
         command
     }
 
-    pub fn get_help(ruby_version: &str, rails_version: &str) -> Command {
+    pub fn get_help(ruby_version: &str, rails_version: Option<&str>) -> Command {
         let mut command = Self::run();
 
         Self::set_image_name(&mut command, ruby_version, rails_version);
@@ -81,8 +87,16 @@ impl DockerClient {
             .args(["-w", current_dir]);
     }
 
-    fn set_image_name(command: &mut Command, ruby_version: &str, rails_version: &str) {
-        command.arg(format!("rails-new-{}-{}", ruby_version, rails_version));
+    fn set_image_name(
+        command: &mut Command,
+        ruby_version: &str,
+        maybe_rails_version: Option<&str>,
+    ) {
+        if let Some(rails_version) = maybe_rails_version {
+            command.arg(format!("rails-new-{}-{}", ruby_version, rails_version));
+        } else {
+            command.arg(format!("rails-new-{}", ruby_version));
+        }
     }
 
     fn set_rails_new(command: &mut Command, args: Vec<String>) {
@@ -121,7 +135,7 @@ mod tests {
 
     #[test]
     fn build_image() {
-        let command = DockerClient::build_image("3.2.3", "7.1.3", None, None, false);
+        let command = DockerClient::build_image("3.2.3", Some("7.1.3"), None, None, false);
 
         assert_eq!(command.get_program(), "docker");
 
@@ -144,7 +158,7 @@ mod tests {
 
     #[test]
     fn build_image_with_user_id() {
-        let command = DockerClient::build_image("3.2.3", "7.1.3", Some(1000), None, false);
+        let command = DockerClient::build_image("3.2.3", Some("7.1.3"), Some(1000), None, false);
 
         assert_eq!(command.get_program(), "docker");
 
@@ -169,7 +183,7 @@ mod tests {
 
     #[test]
     fn build_image_with_group_id() {
-        let command = DockerClient::build_image("3.2.3", "7.1.3", None, Some(1000), false);
+        let command = DockerClient::build_image("3.2.3", Some("7.1.3"), None, Some(1000), false);
 
         assert_eq!(command.get_program(), "docker");
 
@@ -194,7 +208,7 @@ mod tests {
 
     #[test]
     fn run_image() {
-        let command = DockerClient::run_image("3.2.3", "7.1.3", vec!["my_app".to_string()]);
+        let command = DockerClient::run_image("3.2.3", Some("7.1.3"), vec!["my_app".to_string()]);
 
         assert_eq!(command.get_program(), "docker");
 
@@ -223,7 +237,7 @@ mod tests {
 
     #[test]
     fn get_help() {
-        let command = DockerClient::get_help("3.2.3", "7.1.3");
+        let command = DockerClient::get_help("3.2.3", Some("7.1.3"));
 
         assert_eq!(command.get_program(), "docker");
 

src/main.rs

@@ -18,14 +18,14 @@ fn main() {
     let cli = Cli::parse();
 
     let ruby_version = cli.ruby_version;
-    let rails_version = cli.rails_version;
+    let rails_version = cli.rails_version.as_deref();
     let rebuild = cli.rebuild;
 
     // Run docker build --build-arg RUBY_VERSION=$RUBY_VERSION --build-arg RAILS_VERSION=$RAILS_VERSION -t rails-new-$RUBY_VERSION-$RAILS_VERSION
     // passing the content of DOCKERFILE to the command stdin
     let mut child = DockerClient::build_image(
         &ruby_version,
-        &rails_version,
+        rails_version,
         os_specific::get_user_id(),
         os_specific::get_group_id(),
         rebuild,
@@ -46,12 +46,12 @@ fn main() {
 
     match &cli.command {
         Some(Commands::RailsHelp {}) => {
-            command = DockerClient::get_help(&ruby_version, &rails_version)
+            command = DockerClient::get_help(&ruby_version, rails_version)
         }
 
         None => {
             // Run the image with docker run -v $(pwd):/$(pwd) -w $(pwd) rails-new-$RUBY_VERSION-$RAILS_VERSION rails new $@
-            command = DockerClient::run_image(&ruby_version, &rails_version, cli.args)
+            command = DockerClient::run_image(&ruby_version, rails_version, cli.args)
         }
     }
 

src/rails_new.rs

@@ -8,8 +8,8 @@ pub struct Cli {
     pub args: Vec<String>,
     #[clap(long, short = 'u', default_value = "3.4.1")]
     pub ruby_version: String,
-    #[clap(long, short = 'r', default_value = "8.0.1")]
-    pub rails_version: String,
+    #[clap(long, short = 'r')]
+    pub rails_version: Option<String>,
     #[clap(long)]
     pub rebuild: bool,
 
@@ -54,10 +54,8 @@ mod tests {
         let m = Cli::command().get_matches_from(vec!["rails-new", "my_app"]);
 
         let ruby_version = m.get_one::<String>("ruby_version").unwrap();
-        let rails_version = m.get_one::<String>("rails_version").unwrap();
 
         assert_eq!(ruby_version, "3.4.1");
-        assert_eq!(rails_version, "8.0.1");
 
         Ok(())
     }