Add nested lxc test to fully test the lxc_ssh.py plugin from ssh to lxc

[dmp1ce]

Add nested lxc test thanks to the work of @AnvithLobo. This should fully test the lxc_ssh.py plugin.

I'm not sure the previous tests were testing lxc_ssh.py at all to be honest.

[AnvithLobo]

is there any reason to have this test ?

ansible-lxc-ssh/tests/test-lxc_ssh.yml

Lines 50 to 94 in 3e378a4

	- hosts: all
	become: no
	gather_facts: True
	any_errors_fatal: True
	force_handlers: True
	tasks:
	- name: ping VM
	ping:
	- name: Test empty file
	include_tasks:
	file: "test-copy-fetch.yml"
	vars:
	filename: "test_empty"
	- name: Test 1000 Bytes file
	include_tasks:
	file: "test-copy-fetch.yml"
	vars:
	filename: "test_1000b"
	- name: Test 5MB file
	include_tasks:
	file: "test-copy-fetch.yml"
	vars:
	filename: "test_5mb"
	- name: Test 50MB file
	include_tasks:
	file: "test-copy-fetch.yml"
	vars:
	filename: "test_50mb"
	- name: Test command
	include_tasks:
	file: "test-command.yml"
	vars:
	expect_user: "root"
	- name: Test template
	include_tasks:
	file: "test-template.yml"
	vars:
	filename: "test-template"

Since asible_user is by default root when you attach lxc. So become: yes / become: yes should not have any different effect here

[dmp1ce]

I'm not sure. Does become do anything with the plugin?

[AnvithLobo]

I'm not sure. Does become do anything with the plugin?

nope nothing since the default container user is root when you exec commands with lxc . I'm not yet sure if become_user works either. Still have to test that

[dmp1ce]

Well, I'm happy to remove it or leave it in for future use.

[dmp1ce]

I removed the become: no section.

[andreasscherbaum]

@dmp1ce All the new tests are failing with ssh key problem.

[dmp1ce]

@dmp1ce All the new tests are failing with ssh key problem.

That's right! Because the tests from this PR correctly show the state of lxc_ssh plugin. The #41 PR fixed the plugin so the tests pass.

[andreasscherbaum]

Based on discussion here I'm going to merge this PR, and then re-test #38.