feature: use file-pickers instead of text inputs (#168)

Author: ramonvermeulen

README.md

@@ -53,6 +53,11 @@ please refer to the [**official documentation**](https://www.jetbrains.com/help/
 ## Setting environment variables
 If your dbt project requires environment variables (e.g. `DBT_SOME_ENV_VAR`), please set them via the plugin settings.
 `File` > `Settings` > `dbtToolkit` > `Environment Variables`
+
+There is also support for `.env` files, the plugin automatically checks for a `.env` file in the root of your project.
+Via the plugin settings, you can also specify a custom path to the `.env` file.
+> _Note: The global plugin environment variables will overwrite any environment variable with the same key in your 
+> `.env` files_
 <!-- Plugin description end -->
 
 ## Feedback and Contributions

src/main/kotlin/com/github/ramonvermeulen/dbtToolkit/ui/settings/DbtToolkitSettingsConfigurable.kt

@@ -2,10 +2,11 @@ package com.github.ramonvermeulen.dbtToolkit.ui.settings
 
 import com.github.ramonvermeulen.dbtToolkit.services.DbtToolkitSettingsService
 import com.intellij.openapi.components.service
+import com.intellij.openapi.fileChooser.FileChooserDescriptorFactory
 import com.intellij.openapi.options.Configurable
 import com.intellij.openapi.project.Project
+import com.intellij.openapi.ui.TextFieldWithBrowseButton
 import com.intellij.ui.JBIntSpinner
-import com.intellij.ui.components.JBTextField
 import com.intellij.ui.table.JBTable
 import com.intellij.util.ui.JBUI
 import java.awt.BorderLayout
@@ -23,13 +24,34 @@ import javax.swing.table.DefaultTableModel
 
 class DbtToolkitSettingsConfigurable(project: Project) : Configurable {
     private var dbtToolkitSettingsService = project.service<DbtToolkitSettingsService>()
-    private var dbtProjectDirField = JBTextField()
-    private var dbtTargetDirField = JBTextField()
-    private var dotEnvFilePathField = JBTextField()
+    private var dbtProjectDirField = TextFieldWithBrowseButton()
+    private var dbtTargetDirField = TextFieldWithBrowseButton()
+    private var dotEnvFilePathField = TextFieldWithBrowseButton()
     private var dbtCommandTimeoutField = JBIntSpinner(0, 0, 3600)
     private var envVarsTable = JBTable()
     private var settingsPanel = JPanel()
 
+    init {
+        dbtProjectDirField.addBrowseFolderListener(
+            "Select dbt Project Directory",
+            null,
+            project,
+            FileChooserDescriptorFactory.createSingleFolderDescriptor(),
+        )
+        dbtTargetDirField.addBrowseFolderListener(
+            "Select dbt Target Directory",
+            null,
+            project,
+            FileChooserDescriptorFactory.createSingleFolderDescriptor(),
+        )
+        dotEnvFilePathField.addBrowseFolderListener(
+            "Select .env File",
+            null,
+            project,
+            FileChooserDescriptorFactory.createSingleFileDescriptor(),
+        )
+    }
+
     override fun getDisplayName(): String {
         return "dbtToolkit Settings"
     }
@@ -113,8 +135,8 @@ class DbtToolkitSettingsConfigurable(project: Project) : Configurable {
     }
 
     override fun reset() {
-        dbtProjectDirField.text = dbtToolkitSettingsService.state.dbtProjectsDir
-        dbtTargetDirField.text = dbtToolkitSettingsService.state.dbtTargetDir
+        dbtProjectDirField.text = dbtToolkitSettingsService.state.settingsDbtProjectDir
+        dbtTargetDirField.text = dbtToolkitSettingsService.state.settingsDbtTargetDir
         dbtCommandTimeoutField.value = dbtToolkitSettingsService.state.settingsDbtCommandTimeout
         dotEnvFilePathField.text = dbtToolkitSettingsService.state.settingsDotEnvFilePath
 

src/test/kotlin/com/github/ramonvermeulen/dbtToolkit/ui/settings/DbtToolkitSettingsConfigurableTest.kt

@@ -2,11 +2,11 @@ package com.github.ramonvermeulen.dbtToolkit.ui.settings
 
 import com.github.ramonvermeulen.dbtToolkit.services.DbtToolkitSettingsService
 import com.intellij.openapi.components.service
+import com.intellij.openapi.ui.TextFieldWithBrowseButton
 import com.intellij.testFramework.LightPlatformTestCase
 import junit.framework.TestCase
 import javax.swing.JComponent
 import javax.swing.JPanel
-import javax.swing.JTextField
 
 // TODO(ramon) for UI tests, take a lookt at https://github.com/JetBrains/intellij-ui-test-robot
 class DbtToolkitSettingsConfigurableTest : LightPlatformTestCase() {
@@ -29,17 +29,17 @@ class DbtToolkitSettingsConfigurableTest : LightPlatformTestCase() {
     }
 
     fun `test isModified with some modifications`() {
-        val dbtProjectsDirField = (mainSwingComponent.getComponent(0) as JPanel).getComponent(1) as JTextField
+        val dbtProjectsDirField = (mainSwingComponent.getComponent(0) as JPanel).getComponent(1) as TextFieldWithBrowseButton
         dbtProjectsDirField.text = "some text"
-        val dbtDotEnvDirField = (mainSwingComponent.getComponent(0) as JPanel).getComponent(5) as JTextField
+        val dbtDotEnvDirField = (mainSwingComponent.getComponent(0) as JPanel).getComponent(5) as TextFieldWithBrowseButton
         dbtDotEnvDirField.text = "/foo/bar"
         TestCase.assertTrue(configurable.isModified())
     }
 
     fun `test apply`() {
-        val dbtProjectsDirField = (mainSwingComponent.getComponent(0) as JPanel).getComponent(1) as JTextField
+        val dbtProjectsDirField = (mainSwingComponent.getComponent(0) as JPanel).getComponent(1) as TextFieldWithBrowseButton
         dbtProjectsDirField.text = "some text"
-        val dbtDotEnvDirField = (mainSwingComponent.getComponent(0) as JPanel).getComponent(5) as JTextField
+        val dbtDotEnvDirField = (mainSwingComponent.getComponent(0) as JPanel).getComponent(5) as TextFieldWithBrowseButton
         dbtDotEnvDirField.text = "/foo/bar"
         configurable.apply()
         TestCase.assertEquals("some text", settingsService.state.settingsDbtProjectDir)