refactor(_ip_addresses): rename to _comp_ip_addresses

scop · scop · commit a5c7e843243e · 2023-05-27T13:15:22.000+03:00
Per current naming convention.
diff --git a/bash_completion b/bash_completion
@@ -1509,8 +1509,7 @@ _comp_configured_interfaces()
 # -6: IPv6 addresses only
 # -a: All addresses
 #
-# TODO:API: rename per conventions
-_ip_addresses()
+_comp_ip_addresses()
 {
     local n
     case ${1-} in
diff --git a/bash_completion.d/000_bash_completion_compat.bash b/bash_completion.d/000_bash_completion_compat.bash
@@ -15,6 +15,7 @@ _comp_deprecate_func 2.12 __ltrim_colon_completions _comp_ltrim_colon_completion
 _comp_deprecate_func 2.12 _signals _comp_signals
 _comp_deprecate_func 2.12 _mac_addresses _comp_mac_addresses
 _comp_deprecate_func 2.12 _configured_interfaces _comp_configured_interfaces
+_comp_deprecate_func 2.12 _ip_addresses _comp_ip_addresses
 
 # Backwards compatibility for compat completions that use have().
 # @deprecated 1.90 should no longer be used; generally not needed with
diff --git a/completions/arping b/completions/arping
@@ -14,7 +14,7 @@ _comp_cmd_arping()
             return
             ;;
         -*s)
-            _ip_addresses
+            _comp_ip_addresses
             return
             ;;
     esac
diff --git a/completions/curl b/completions/curl
@@ -67,7 +67,7 @@ _comp_cmd_curl()
             return
             ;;
         --dns-ipv[46]-addr)
-            _ip_addresses -"${prev:9:1}"
+            _comp_ip_addresses -"${prev:9:1}"
             return
             ;;
         --dns-servers | --noproxy)
@@ -87,7 +87,7 @@ _comp_cmd_curl()
         --ftp-port | -${noargopts}P)
             _available_interfaces -a
             _known_hosts_real -- "$cur"
-            _ip_addresses -a
+            _comp_ip_addresses -a
             return
             ;;
         --ftp-method)
diff --git a/completions/hddtemp b/completions/hddtemp
@@ -13,7 +13,7 @@ _comp_cmd_hddtemp()
             return
             ;;
         --listen | -${noargopts}l)
-            _ip_addresses
+            _comp_ip_addresses
             return
             ;;
         --unit | -${noargopts}u)
diff --git a/completions/iperf b/completions/iperf
@@ -36,7 +36,7 @@ _comp_cmd_iperf()
             ;;
         --bind | -${noargopts}B)
             _available_interfaces -a
-            _ip_addresses -a
+            _comp_ip_addresses -a
             return
             ;;
         --bind-dev)
diff --git a/completions/links b/completions/links
@@ -42,11 +42,11 @@ _comp_cmd_links()
             return
             ;;
         -bind-address)
-            _ip_addresses
+            _comp_ip_addresses
             return
             ;;
         -bind-address-ipv6)
-            _ip_addresses -6
+            _comp_ip_addresses -6
             return
             ;;
         -async-dns | -download-utime | -aggressive-cache | -only-proxies | \
diff --git a/completions/nc b/completions/nc
@@ -11,9 +11,9 @@ _comp_cmd_nc()
             ;;
         -*s)
             if [[ ${words[*]} == *-6* ]]; then
-                _ip_addresses -6
+                _comp_ip_addresses -6
             else
-                _ip_addresses
+                _comp_ip_addresses
             fi
             return
             ;;
diff --git a/completions/ping b/completions/ping
@@ -40,7 +40,7 @@ _comp_cmd_ping()
             ;;
         -*S)
             # Socket sndbuf in Linux, source IP in FreeBSD
-            [[ $OSTYPE == *bsd* ]] && _ip_addresses
+            [[ $OSTYPE == *bsd* ]] && _comp_ip_addresses
             return
             ;;
         -*T)
diff --git a/completions/ssh b/completions/ssh
@@ -132,7 +132,7 @@ _comp_cmd_ssh__suboption()
             _comp_compgen -- -W 'any inet inet6'
             ;;
         bindaddress)
-            _ip_addresses
+            _comp_ip_addresses
             ;;
         canonicalizehostname)
             _comp_compgen -- -W 'yes no always'
@@ -304,7 +304,7 @@ _comp_cmd_ssh()
             return
             ;;
         -*b)
-            _ip_addresses
+            _comp_ip_addresses
             return
             ;;
         -*c)
diff --git a/completions/wget b/completions/wget
@@ -16,7 +16,7 @@ _comp_cmd_wget()
             return
             ;;
         --bind-address)
-            _ip_addresses
+            _comp_ip_addresses
             return
             ;;
         --domains | --exclude-domains | -${noargopts}D)
diff --git a/test/t/unit/test_unit_ip_addresses.py b/test/t/unit/test_unit_ip_addresses.py
@@ -10,40 +10,40 @@ def functions(self, request, bash):
         assert_bash_exec(
             bash,
             "_ia() { local cur;_comp_get_words cur;"
-            "unset -v COMPREPLY;_ip_addresses; }",
+            "unset -v COMPREPLY;_comp_ip_addresses; }",
         )
         assert_bash_exec(bash, "complete -F _ia ia")
         assert_bash_exec(
             bash,
             "_iaa() { local cur;_comp_get_words cur;"
-            "unset -v COMPREPLY;_ip_addresses -a; }",
+            "unset -v COMPREPLY;_comp_ip_addresses -a; }",
         )
         assert_bash_exec(bash, "complete -F _iaa iaa")
         assert_bash_exec(
             bash,
             " _ia6() { local cur;_comp_get_words cur;"
-            "unset -v COMPREPLY;_ip_addresses -6; }",
+            "unset -v COMPREPLY;_comp_ip_addresses -6; }",
         )
         assert_bash_exec(bash, "complete -F _ia6 ia6")
 
     def test_1(self, bash):
-        assert_bash_exec(bash, "_ip_addresses")
+        assert_bash_exec(bash, "_comp_ip_addresses")
 
     @pytest.mark.complete("iaa ")
     def test_2(self, functions, completion):
-        """_ip_addresses -a should complete ip addresses."""
+        """_comp_ip_addresses -a should complete ip addresses."""
         assert completion
         assert all("." in x or ":" in x for x in completion)
 
     @pytest.mark.complete("ia ")
     def test_3(self, functions, completion):
-        """_ip_addresses should complete ipv4 addresses."""
+        """_comp_ip_addresses should complete ipv4 addresses."""
         assert completion
         assert all("." in x for x in completion)
 
     @pytest.mark.xfail(in_container(), reason="Probably fails in a container")
     @pytest.mark.complete("ia6 ")
     def test_4(self, functions, completion):
-        """_ip_addresses -6 should complete ipv6 addresses."""
+        """_comp_ip_addresses -6 should complete ipv6 addresses."""
         assert completion
         assert all(":" in x for x in completion)

Original file line number	Diff line number	Diff line change
`@@ -1509,8 +1509,7 @@ _comp_configured_interfaces()`
`1509`	`1509`	`# -6: IPv6 addresses only`
`1510`	`1510`	`# -a: All addresses`
`1511`	`1511`	`#`
`1512`		`-# TODO:API: rename per conventions`
`1513`		`-_ip_addresses()`
	`1512`	`+_comp_ip_addresses()`
`1514`	`1513`	`{`
`1515`	`1514`	`local n`
`1516`	`1515`	`case ${1-} in`