fix: removed some deprecated methods

Author: fstagni

src/DIRAC/Core/Base/Script.py

@@ -69,7 +69,7 @@ def __call__(self, func=None):
                 "Invalid dirac- console_scripts entry_point: "
                 + repr(entrypoint)
                 + "\n"
-                + "All dirac- console_scripts should be wrapped in the DiracScript "
+                + "All dirac- console_scripts should be wrapped in the Script "
                 + "decorator to ensure extension overlays are applied correctly."
             )
         return entrypointFunc._func()

src/DIRAC/Core/Security/m2crypto/X509Chain.py

@@ -20,7 +20,7 @@
 
 from DIRAC import S_OK, S_ERROR
 from DIRAC.Core.Utilities import DErrno
-from DIRAC.Core.Utilities.Decorators import executeOnlyIf, deprecated
+from DIRAC.Core.Utilities.Decorators import executeOnlyIf
 from DIRAC.ConfigurationSystem.Client.Helpers import Registry
 from DIRAC.Core.Security.m2crypto import PROXY_OID, LIMITED_PROXY_OID, DIRAC_GROUP_OID, DEFAULT_PROXY_STRENGTH
 from DIRAC.Core.Security.m2crypto.X509Certificate import X509Certificate
@@ -165,22 +165,6 @@ def __init__(self, certList=False, keyObj=False):
         if keyObj:
             self._keyObj = keyObj
 
-    @classmethod
-    @deprecated("Use loadChainFromFile instead", onlyOnce=True)
-    def instanceFromFile(cls, chainLocation):
-        """Class method to generate a X509Chain from a file
-
-        :param chainLocation: path to the file
-
-        :returns: S_OK(X509Chain)
-        """
-        chain = cls()
-        result = chain.loadChainFromFile(chainLocation)
-        if not result["OK"]:
-            return result
-
-        return S_OK(chain)
-
     @staticmethod
     def generateX509ChainFromSSLConnection(sslConnection):
         """Returns an instance of X509Chain from the SSL connection
@@ -393,24 +377,6 @@ def getIssuerCert(self):
             return S_OK(self._certList[self.__firstProxyStep + 1])
         return S_OK(self._certList[-1])
 
-    @deprecated("Only here for compatibility reason", onlyOnce=True)
-    @needPKey
-    def getPKeyObj(self):
-        """
-        Get the pkey obj
-
-        :returns: ~M2Crypto.EVP.PKey object
-        """
-        return S_OK(self._keyObj)
-
-    @deprecated("Only here for compatibility reason")
-    @needCertList
-    def getCertList(self):
-        """
-        Get the cert list
-        """
-        return S_OK(self._certList)
-
     @needCertList
     def getNumCertsInChain(self):
         """

src/DIRAC/Core/Security/test/Test_X509Chain.py

@@ -155,33 +155,13 @@ def test_loadChainFromString_non_pem(get_X509Chain_class):
     assert res["Errno"] == EX509
 
 
-@parametrize("key_file", CERTKEYS)
-def test_init_with_key(key_file, get_X509Chain_class):
-    """Test init with key object as argument and check the content"""
-
-    chain1 = get_X509Chain_class()
-    chain1.loadKeyFromFile(key_file)
-
-    # Get the key and check the number of bits
-    keyObj = chain1.getPKeyObj()["Value"]
-
-    chain2 = get_X509Chain_class(keyObj=keyObj)
-    assert chain1.dumpPKeyToString() == chain2.dumpPKeyToString()
-    # Careful ! The two keys are the same object
-    assert chain2.getPKeyObj()["Value"] is keyObj
-
-
 @parametrize("key_file", CERTKEYS)
 def test_privatekey_without_password(key_file, get_X509Chain_class):
     """Test loading a key from a file, retrieve the object and check the content"""
 
     X509Chain = get_X509Chain_class()
     res = X509Chain.loadKeyFromFile(key_file)
     assert res["OK"]
-    # Get the key and check the number of bits
-    res = X509Chain.getPKeyObj()
-    assert res["OK"]
-    assert res["Value"].size() == 512
 
     # Check that the content of the object is correct.
     # CAUTION ! The object is PKCS8, while the file contains PKCS1.
@@ -195,10 +175,6 @@ def test_privatekey_with_password(get_X509Chain_class):
     X509Chain = get_X509Chain_class()
     res = X509Chain.loadKeyFromFile(ENCRYPTEDKEY, password=ENCRYPTEDKEYPASS)
     assert res["OK"]
-    # Get the key and check the number of bits
-    res = X509Chain.getPKeyObj()
-    assert res["OK"]
-    assert res["Value"].size() == 512
 
 
 def test_privatekey_with_wrong_password(get_X509Chain_class):
@@ -233,17 +209,6 @@ def test_getCertInChain_too_far(get_X509Chain_class):
         x509Chain.getCertInChain(1)
 
 
-@parametrize("cert_file", CERTS)
-def test_getCertList(cert_file, get_X509Chain_class):
-    """ " Load a chain, and get its length."""
-    x509Chain = get_X509Chain_class()
-    x509Chain.loadChainFromFile(cert_file)
-    # For a certificate, there should be only 1 certificate in the chain
-
-    assert len(x509Chain.getCertList()["Value"]) == 1
-    assert len(x509Chain.getCertList()["Value"]) == x509Chain.getNumCertsInChain()["Value"]
-
-
 @parametrize("cert_file", CERTS)
 def test_certProperties(cert_file, get_X509Chain_class):
     """Try on a certificate if it is a proxy, limited proxy, VOMS, valid proxy, rfc"""

src/DIRAC/Core/Utilities/Os.py

@@ -3,10 +3,8 @@
    by default on Error they return None
 """
 import os
-import shutil
 
 import DIRAC
-from DIRAC.Core.Utilities.Decorators import deprecated
 from DIRAC.Core.Utilities.Subprocess import shellCall, systemCall
 from DIRAC.Core.Utilities import List
 
@@ -145,8 +143,3 @@ def sourceEnv(timeout, cmdTuple, inputEnv=None):
     result["stderr"] = stderr
 
     return result
-
-
-@deprecated("Will be removed in DIRAC 8.1", onlyOnce=True)
-def which(executable):
-    return shutil.which(executable)

src/DIRAC/Resources/ProxyProvider/test/Test_DIRACCAProxyProvider.py

@@ -105,7 +105,6 @@ def check(proxyStr, proxyProvider, name):
             for result in [
                 chain.getRemainingSecs(),
                 chain.getIssuerCert(),
-                chain.getPKeyObj(),
                 chain.getCertList(),
                 chain.getNumCertsInChain(),
                 chain.generateProxyToString(3600),