Merge pull request dragonflyoss#841 from yeya24/cleanup-dfget-fields

cleanup some unused dfget config fields

cmd/dfget/app/root_test.go

@@ -45,11 +45,9 @@ func (suit *dfgetSuit) Test_initFlagsNoArguments() {
 	suit.Equal(cfg.TotalLimit, 0)
 	suit.Equal(cfg.Notbs, false)
 	suit.Equal(cfg.DFDaemon, false)
-	suit.Equal(cfg.Version, false)
 	suit.Equal(cfg.ShowBar, false)
 	suit.Equal(cfg.Console, false)
 	suit.Equal(cfg.Verbose, false)
-	suit.Equal(cfg.Help, false)
 	suit.Equal(cfg.URL, "")
 }
 

dfget/config/config.go

@@ -191,9 +191,6 @@ type Config struct {
 	// Insecure indicates whether skip secure verify when supernode interact with the source.
 	Insecure bool `json:"insecure,omitempty"`
 
-	// Version show version.
-	Version bool `json:"version,omitempty"`
-
 	// ShowBar show progress bar, it's conflict with `--console`.
 	ShowBar bool `json:"showBar,omitempty"`
 
@@ -204,9 +201,6 @@ type Config struct {
 	// If set true, log level will be 'debug'.
 	Verbose bool `json:"verbose,omitempty"`
 
-	// Help show help information.
-	Help bool `json:"help,omitempty"`
-
 	// ClientQueueSize is the size of client queue
 	// which controls the number of pieces that can be processed simultaneously.
 	// It is only useful when the pattern not equals "source".

dfget/config/config_test.go

@@ -58,9 +58,8 @@ func (suite *ConfigSuite) TestConfig_String(c *check.C) {
 	c.Assert(strings.Contains(cfg.String(), expected), check.Equals, true)
 	cfg.LocalLimit = 20971520
 	cfg.Pattern = "p2p"
-	cfg.Version = true
 	expected = "\"url\":\"\",\"output\":\"\",\"localLimit\":20971520," +
-		"\"pattern\":\"p2p\",\"version\":true"
+		"\"pattern\":\"p2p\""
 	c.Assert(strings.Contains(cfg.String(), expected), check.Equals, true)
 }