For:
[ 13-test-signal-compare_obj.diff ]
[ 17-signal-rule-cleanup.diff ]
[ 21-test-signal-rename-tests.diff ]

Acked-by: Kshitij Gupta <kgupta8...@gmail.com>

On Tue, Nov 17, 2015 at 2:23 AM, Christian Boltz <appar...@cboltz.de> wrote:

> Hello,
>
> Am Freitag, 23. Oktober 2015 schrieb Christian Boltz:
> > this patch adds the SignalRule and SignalRuleset classes
>
> Here's the third patch on top:
>
> [patch] Rename some test-signal tests
>
> signal tests with "net" in their name don't look like a good idea ;-)
>
>
> I'll commit this together with 07-add-SignalRule-and-SignalRuleset.diff
> (and slightly adjust 12-logprof-signal.diff for the changed name)
>
>
> [ 21-test-signal-rename-tests.diff ]
>
> === modified file ./utils/test/test-signal.py
> --- utils/test/test-signal.py   2015-11-16 21:36:19.467063449 +0100
> +++ utils/test/test-signal.py   2015-11-16 00:14:05.371336371 +0100
> @@ -88,7 +88,7 @@
>              SignalRule.parse(rawrule)
>
>  class SignalTestParseFromLog(SignalTest):
> -    def test_net_from_log(self):
> +    def test_signal_from_log(self):
>          parser = ReadLog('', '', '', '', '')
>          event = 'type=AVC msg=audit(1409438250.564:201):
> apparmor="DENIED" operation="signal" profile="/usr/bin/pulseaudio" pid=2531
> comm="pulseaudio" requested_mask="send" denied_mask="send" signal=term
> peer="/usr/bin/pulseaudio///usr/lib/pulseaudio/pulse/gconf-helper"'
>
> @@ -187,7 +187,7 @@
>
>          self.assertIsNone(obj, 'SignalRule handed back an object
> unexpectedly')
>
> -    def test_invalid_net_missing_comma(self):
> +    def test_invalid_signal_missing_comma(self):
>          self._check_invalid_rawrule('signal')  # missing comma
>
>      def test_invalid_non_SignalRule(self):
>
>
>
>
>
> Regards,
>
> Christian Boltz
> --
> > Womit erstellt ihr so eure Homepages?
> mit vim *g*.  Wobei es Leute gibt, die tatsächlich behaupten, das soll
> auch mit diesem Betriebssystem - wie heißt es doch gleich - *äh* Emacs
> gehen. <SCNR>   [> Bernd Stäglich und Philipp Zacharias in suse-linux]
>
>
> --
> AppArmor mailing list
> AppArmor@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/apparmor
>



-- 
Regards,

Kshitij Gupta
-- 
AppArmor mailing list
AppArmor@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to